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

Import local schema to avoid DNS lookup in build #5372

Merged
merged 1 commit into from
Jan 27, 2023

Conversation

jgneff
Copy link
Contributor

@jgneff jgneff commented Jan 26, 2023

This is the same fix as the one made to jakartaee_9.xsd in NetBeans 16 for #4920, but to the new file jakartaee_10.xsd in NetBeans 17. It allows NetBeans to be built in environments with very strict firewalls, such as Launchpad, by removing the only DNS lookup during the build.

I'd like to make sure NetBeans can build on Launchpad because that's the easiest way to publish Snap packages of NetBeans for all supported Linux architectures: amd64, arm64, armhf, and i386.

@matthiasblaesing
Copy link
Contributor

Thanks for catching this, the change looks good. This needs to be cherry-picked onto delivery. Changes for NB17 need to come from branch delivery, changes in master go to NB18.

@jgneff
Copy link
Contributor Author

jgneff commented Jan 26, 2023

Thanks, Matthias. I wasn't sure whether I was too late for NetBeans 17. Is that something I should do (cherry-pick this onto delivery)?

@matthiasblaesing matthiasblaesing added this to the NB17 milestone Jan 26, 2023
@matthiasblaesing matthiasblaesing changed the base branch from master to delivery January 26, 2023 20:04
@neilcsmith-net neilcsmith-net added the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Jan 26, 2023
@matthiasblaesing
Copy link
Contributor

There might or might not be a new RC, if there is this should be in. I added it to the NB17 milestone and changed the target branch to delivery, but you can see in the list of commits, that merging as is would cause multiple commits to be pulled in.

There are two options:

a) You create a new pull request that is based on delivery and opened with target delivery or
b) You feel happy with the git tools and do basicly this:

# Ensure the branch this PR is based on is active
git checkout import-local-schema
# Move it to a different name
git branch -m import-local-schema-on-master
# Create a new local branch with the old name
# (0e504e5911846f4f66b3d97cb5d1c884b83a50ab is current HEAD of `delivery`)
# maybe you need to `git fetch` from the github repository of apache netbeans
git checkout -b import-local-schema 0e504e5911846f4f66b3d97cb5d1c884b83a50ab
# Cherry pick your commit into the new branch
git cherry-pick 2dce1801e795731248679ca061315db7e2fcd6fa
# Force push this branch into the original branch that created this PR (this assumes you named your upstream origin)
git push -f -u origin import-local-schema

@neilcsmith-net
Copy link
Member

You can rebase using

git rebase HEAD~1 --onto delivery then force push.

Label just until this is done.

@matthiasblaesing
Copy link
Contributor

The intermediate test failure was a flaky commit validation I saw in the past and is fixed after restarting.

@mbien mbien added Java EE/Jakarta EE [ci] enable enterprise job enterprise [ci] enable enterprise job labels Jan 26, 2023
@jgneff
Copy link
Contributor Author

jgneff commented Jan 26, 2023

The following appears to have worked!

$ git switch -c delivery upstream/delivery
$ git switch import-local-schema
$ git rebase --onto delivery import-local-schema~ import-local-schema
$ git push --force

@mbien
Copy link
Member

mbien commented Jan 26, 2023

added labels otherwise this would show up under the do-not-merge category on the release notes :P
looks like its on the right branch too now.

to avoid having to look up the hash in the log, i often use a graphical tool for it like gitg. Where i can simply right click on a commit and select cherry pick onto which is super convenient.

but the basic concept is simply:

git checkout copy_of_delivery_branch
git cherry-pick somehash

thats it.

@neilcsmith-net neilcsmith-net removed the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Jan 26, 2023
@neilcsmith-net
Copy link
Member

Label removed. Thanks @jgneff Will be included in 17-rc3.

There might or might not be a new RC, if there is this should be in.

We will always have at least 3.

Where i can simply right click on a commit and select cherry pick onto which is super convenient.

Yes, but still not as convenient as rebase --onto in my opinion, given it does what it says and uses the existing branch.

@neilcsmith-net neilcsmith-net merged commit b68e3fb into apache:delivery Jan 27, 2023
@jgneff jgneff deleted the import-local-schema branch June 28, 2023 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enterprise [ci] enable enterprise job Java EE/Jakarta EE [ci] enable enterprise job
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants