Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Snapshot of APIs as of NetBeans 19 #6252

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

neilcsmith-net
Copy link
Member

@neilcsmith-net neilcsmith-net commented Jul 25, 2023

Snapshot of APIs as of NetBeans 19. For review purposes only at this stage. Not to be merged as / until release vote completes and updated if necessary.

Built in the usual way ...
# checkout branch (or release tag)
git checkout release190
git clean -dxf
ant build
ant gen-sigtests-release
git add -A
git stash
git checkout master
git checkout -b apis-nb190
git stash pop
git add -A
git commit -m "Snapshot of APIs as of NetBeans 19"
# remove getPeer() calls
find . -name "*.sig" -exec sed -i '/java.awt.peer.ComponentPeer/{N;d;}' {} \;
# check git diff
git add -A
git commit --amend

Filtered diff created using -

curl -L https://github.com/apache/netbeans/pull/<PR_ID>.diff | filterdiff -#2- > filtered-apis-nb190.txt

Filtered diff : filtered-apis-nb190.txt

@neilcsmith-net neilcsmith-net added do not merge Don't merge this PR, it is not ready or just demonstration purposes. Release process PRs (eg. versions, sync) that are part of the release process and can be ignored in release notes. labels Jul 25, 2023
@ebarboni
Copy link
Contributor

LGTM huge changes with index bump also.

@ebarboni
Copy link
Contributor

ebarboni commented Sep 1, 2023

I see using cli that

both modified: java/java.source.base/nbproject/org-netbeans-modules-java-source-base.sig

I'm not sure what has to be done

@neilcsmith-net
Copy link
Member Author

@ebarboni I can handle next week if you want? This needs regenerating too before merge as I think there may be a couple of extra minor changes from the first RC.

@neilcsmith-net
Copy link
Member Author

neilcsmith-net commented Sep 7, 2023

@ebarboni regenerated and fixed against current master. Just waiting for tests.

In future we might want to consider doing this via delivery, with a second merge commit? The commit is not an accurate snapshot of APIs, as it includes changes to sigfiles from master since branch.

@ebarboni
Copy link
Contributor

ebarboni commented Sep 7, 2023

LGTM by the way. Any workflow that ease is welcome

@neilcsmith-net
Copy link
Member Author

Thanks @ebarboni Tests look good so merging.

Any workflow that ease is welcome

It won't make it any easier! It will just ensure that there is a commit somewhere that actually contains an accurate snapshot.

@neilcsmith-net neilcsmith-net merged commit a60e7cb into apache:master Sep 8, 2023
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Don't merge this PR, it is not ready or just demonstration purposes. Release process PRs (eg. versions, sync) that are part of the release process and can be ignored in release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants