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

List mirrors of updates sites right after the original site #163

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

carandraug
Copy link
Contributor

The order that update sites matter when multiple update sites provide the same files. This is how the Java-8 update site overwrites files provided by the ImageJ and Fiji update sites.

At the moment, the ImageJ, Fiji, and Java-8 mirror site are listed after the original. If someone selects the ImageJ and Fiji mirrors but not the Java-8 mirror, then the Java-8 overwrite is replaced by the mirrors. To avoid this, this commit moves the ImageJ and Fiji update sites under the original update site. See
https://forum.image.sc/t/errors-with-european-mirrors-for-fiji-and-imagej/103355


I didn't actually check in the code how the updater chooses files, I'm guessing that's how it does it from the behaviour.

The order that update sites matter when multiple update sites provide
the same files.  This is how the Java-8 update site overwrites files
provided by the ImageJ and Fiji update sites.

At the moment, the ImageJ, Fiji, and Java-8 mirror site are listed
after the original.  If someone selects the ImageJ and Fiji mirrors
but not the Java-8 mirror, then the Java-8 overwrite is replaced by
the mirrors.  To avoid this, this commit moves the ImageJ and Fiji
update sites under the original update site.  See
https://forum.image.sc/t/errors-with-european-mirrors-for-fiji-and-imagej/103355
@imagesc-bot
Copy link

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/errors-with-european-mirrors-for-fiji-and-imagej/103355/2

@ctrueden
Copy link
Member

ctrueden commented Oct 8, 2024

@carandraug Good idea!

@ctrueden ctrueden merged commit 3a8bee2 into imagej:main Oct 8, 2024
1 check passed
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