Skip to content

Released 5.0.1 - #299

Merged
dmatej merged 4 commits into
masterfrom
RELEASE_501
May 4, 2026
Merged

Released 5.0.1#299
dmatej merged 4 commits into
masterfrom
RELEASE_501

Conversation

@dmatej

@dmatej dmatej commented May 4, 2026

Copy link
Copy Markdown
Contributor
  • Finally releasing as 5.0.1, because there are no breaking changes, compared to 5.0.0.
  • Old repositories are already removed, so we don't have to disable them
  • Deployment is set correctly now, so we don't need most of the original cfg

- Old repositories are already removed, so we don't have to disable them
- Deployment is set correctly now, so we don't need most of the original cfg

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
@dmatej dmatej added this to the 5.1.0 milestone May 4, 2026
@dmatej dmatej changed the title Adapted to parent project 2.0.2 Released 5.0.1 May 4, 2026
dmatej and others added 3 commits May 4, 2026 09:28
- From now on we can check compatibility with snapshot dependencies if they
  deploy to maven central snapshots. Maven is smart enough to know if it
  should check snapshot of "final" central repository.

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
@dmatej
dmatej marked this pull request as ready for review May 4, 2026 08:03
@dmatej
dmatej requested a review from a team May 4, 2026 08:03
@dmatej
dmatej merged commit 41c41d3 into master May 4, 2026
2 checks passed
@dmatej
dmatej deleted the RELEASE_501 branch May 4, 2026 08:25
@pzygielo

pzygielo commented May 4, 2026

Copy link
Copy Markdown
Contributor
  • Finally releasing as 5.0.1, because there are no breaking changes, compared to 5.0.0.

But this:

by the happy accident of name collision, changed for example

old: parameter org.omg.CORBA.portable.InputStream org.omg.CORBA.LocalObject::_invoke(===org.omg.CORBA.portable.OutputStream===) throws org.omg.CORBA.portable.ApplicationException, org.omg.CORBA.portable.RemarshalException @ com.sun.corba.ee.impl.interceptors.CodecFactoryImpl
new: parameter java.io.InputStream org.omg.CORBA.LocalObject::_invoke(===java.io.OutputStream===) throws org.omg.CORBA.portable.ApplicationException, org.omg.CORBA.portable.RemarshalException @ com.sun.corba.ee.impl.interceptors.CodecFactoryImpl
java.method.parameterTypeChanged: The type of the parameter changed from 'org.omg.CORBA.portable.OutputStream' to 'java.io.OutputStream'.
SOURCE: POTENTIALLY_BREAKING, BINARY: BREAKING

I do not know how public this is

*/
public InputStream _invoke(OutputStream output)
throws ApplicationException, RemarshalException
{
throw new org.omg.CORBA.NO_IMPLEMENT(reason);
}

Looks like the one waiting to be implemented by extending class.

🤔

@pzygielo

pzygielo commented May 4, 2026

Copy link
Copy Markdown
Contributor

Perhaps those types should be restored and 5.0.2 released.

@dmatej

dmatej commented May 4, 2026

Copy link
Copy Markdown
Contributor Author

Aaargh, I tried to check these names for sure manually, but with all those asterisks it was a hard task. Exactly the reason why to avoid asterisks.
Ok, I did not release 8.0.2 yet, I was going to do that now, because the TCK passed. So lets fix this first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants