Skip to content

GeoWebCache update for JAI to ImageN rename of classes and constants#1434

Merged
jodygarnett merged 1 commit into
mainfrom
jai-rename
Oct 5, 2025
Merged

GeoWebCache update for JAI to ImageN rename of classes and constants#1434
jodygarnett merged 1 commit into
mainfrom
jai-rename

Conversation

@jodygarnett

@jodygarnett jodygarnett commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

@aaime

aaime commented Sep 29, 2025

Copy link
Copy Markdown
Member

I cannot build the branch as-is (using renamed ImageN):

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.14.0:compile (default-compile) on project gwc-core: Compilation failure: Compilation failure: 
[ERROR] /home/aaime/devel/git-gwc/geowebcache/core/src/main/java/org/geowebcache/layer/MetaTile.java:[36,26] cannot find symbol
[ERROR]   symbol:   class JAI
[ERROR]   location: package org.eclipse.imagen
[ERROR] /home/aaime/devel/git-gwc/geowebcache/core/src/main/java/org/geowebcache/mime/ImageMime.java:[29,26] cannot find symbol
[ERROR]   symbol:   class JAI
[ERROR]   location: package org.eclipse.imagen
[ERROR] -> [Help 1]
[ERROR] 

I've made the necessary fixes and pushed a commit to this branch.

@jodygarnett

Copy link
Copy Markdown
Contributor Author

Huh, it seems I failed to run the script before making the PR, let me try again and see if it catches this change.

No direct API usage, apparnelty everythign is done via ImageWorker.

Adapt to ImageN API changes
@jodygarnett

Copy link
Copy Markdown
Contributor Author

Huh, it seems I failed to run the script before making the PR, let me try again and see if it catches this change.

Sorry about that did not manage to run the update script on this branch so the PR was incomplete. The update script caught a couple things (comments and strings) that would not of shown up as compile error.

@jodygarnett
jodygarnett merged commit a77b866 into main Oct 5, 2025
1 of 9 checks passed
@jodygarnett
jodygarnett deleted the jai-rename branch October 5, 2025 07:16
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.

2 participants