-
Notifications
You must be signed in to change notification settings - Fork 8
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
Prepare imglib2-ij for modularization #37
Comments
Changing the package name will break all existing documentation and
tutorials online. Is this necessary?
…On Fri, 5 Jan 2024, 16:23 Tobias Pietzsch, ***@***.***> wrote:
We'll start by fixing imglib2-ij.
- Set <package-name>net.imglib2.ij</package-name>.
- Move all classes under the net.imglib2.ij package.
To update dependant projects, we need tooling to
- Find and fix import statements that reference imglib2-ij classes.
- Find and fix fully qualified uses of imglib2-ij classes.
- Create PRs to the dependant projects.
Find and fix import statements that reference imglib2-ij classes.
- Clone all relevant candidate repos.
- List types (and interfaces, and enums) of imglib2-ij.
—
Reply to this email directly, view it on GitHub
<#37>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAMKGTNUJCGDSRTMJSPTATYNASJNAVCNFSM6AAAAABBOU6OBSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3DONRUGM2TGNQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Yes, it is necessary. JPMS requires that there are no split packages between artefacts. Breaking documentation and examples is a very good point. I did not think of that. We should also have scripts to update these. However its more difficult to gather all relevant repositories. (For code, we can at least harvest everything managed by pom-scijava.) Feel free to start putting inks in the comments of this issue! |
Will also brake all user scripts. Most aren't under version control. Might
want to make this a breaking change with a separate Fiji version, leaving
the old untouched and not receiving more updates, to avoid chaos and
confusion.
…On Sun, 7 Jan 2024, 21:47 Tobias Pietzsch, ***@***.***> wrote:
Yes, it is necessary. JPMS requires that there are no split packages
between artefacts.
(The same is true for OSGi. If I remember correctly, KNIME image
processing for example had to work around this by having their own version
of imglib2-ij.)
Breaking documentation and examples is a very good point. I did not think
of that. We should also have scripts to update these. However its more
difficult to gather all relevant repositories. (For code, we can at least
harvest everything managed by pom-scijava.) Feel free to start putting inks
in the comments of this issue!
—
Reply to this email directly, view it on GitHub
<#37 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAMKGVF3SLY4NFJIRHPTC3YNMJYRAVCNFSM6AAAAABBOU6OBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGE4DMMRYGA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Another good point. Maybe another solution might be to have a new artefact ( I would be also very interested in @ctrueden thoughts on this. |
<package-name>net.imglib2.ij</package-name>
.net.imglib2.ij
package.The text was updated successfully, but these errors were encountered: