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

Reinstate support for eXist-db 4.x.x #78

Merged
merged 2 commits into from Jul 5, 2022
Merged

Conversation

adamretter
Copy link
Member

The eXist-db 4.x.x build process requires the public-repo to deliver a Zip file containing the Xar file. This was working in the past. However, it now looks like support for eXist-db 4.x.x was accidentally removed through two refactorings:

  1. 90eb6bb#diff-bc346eefb44d6303a883aca1f15e3f0e68895c005bfa330f2ea00b7bf2cc70f0
  2. b65a476#diff-bc346eefb44d6303a883aca1f15e3f0e68895c005bfa330f2ea00b7bf2cc70f0L1

This PR reinstates support for eXist-db 4.x.x.

modules/get-package.xq Outdated Show resolved Hide resolved
Copy link
Member

@joewiz joewiz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on an eXist 6.1.0 system and confirmed that the request to http://localhost:8080/exist/apps/public-repo/find?abbrev=dashboard&processor=6.0.1-SNAPSHOT&zip=yes causes dashboard-2.0.6.xar.zip to be downloaded, and the zip file itself contains dashboard-2.0.6.xar. From my understanding of the issue, this is the behavior that eXist 4.x's build process relies on to download packages intended for the autodeploy facility.

@joewiz joewiz merged commit 2e38ff6 into master Jul 5, 2022
@adamretter
Copy link
Member Author

Thanks so much @joewiz :-)

@joewiz joewiz deleted the hotfix/exist-4.x.x-support branch January 7, 2023 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants