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

change to repo.osgeo.org #845

Merged
merged 4 commits into from Apr 7, 2020
Merged

Conversation

jodygarnett
Copy link
Contributor

This project makes use of geoserver repository for release publication (mirroring what was done for repo.boundlessgeo.com).

Signed-off-by: Jody Garnett jody.garnett@gmail.com

This project makes use of geoserver repository for release publication (mirroring what was done for repo.boundlessgeo.com).

Signed-off-by: Jody Garnett <jody.garnett@gmail.com>
@jodygarnett
Copy link
Contributor Author

jodygarnett commented Apr 7, 2020

@tbarsballe following on from geotools/geotools#2863 here is a change to geowebcache.

Please note that each project has a different distribution management repository to upload to (in this case geowebcache is sharing with geoserver to better mirror what was done at repo.boundlessgeo.com).

Update: Changed distribution management section to use id "nexus" so build server only has one set of credentials to mange.

By using the same id the build server only has one set of credentials to manage in settings.xml to access osgeo repository.

Signed-off-by: Jody Garnett <jody.garnett@gmail.com>
Copy link
Contributor

@tbarsballe tbarsballe left a comment

Choose a reason for hiding this comment

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

See comments inline. Thanks for switching to using "nexus" in the distribution repository ids

<name>Boundless Maven Repository</name>
<url>https://repo.boundlessgeo.com/main/</url>
<id>osgeo-release</id>
<name>OSGeo Repository</name>
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe use "OSGeo Release Repository: for the name here (and "OSGeo Snapshot Repository" below)

<!-- Contains releases for OSGeo projects: GeoTools, GeoWebCache, GeoServer, ... -->
<!-- Includs third-party-dependences, and caches of: -->
<!-- ucar (https://artifacts.unidata.ucar.edu/content/repositories/unidata-releases) -->
<!-- geosolutions (http://maven.geo-solutions.it/) -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Aren't these only in the release repository, not the snapshot one?

Also, I don't believe GeoWebCache cares about either of those (maybe geosolutions?), so there isn't much of a reason to mention them

@tbarsballe
Copy link
Contributor

GeoTools builds have completed successfully on Jenkins, and geotools-snapshots is now populated with 24-SNAPSHOT on nexus.

All build checks have passed here, and the only issues I had are with docs/naming, so feel free to merge this when ready.

Signed-off-by: Jody Garnett <jody.garnett@gmail.com>
Signed-off-by: Jody Garnett <jody.garnett@gmail.com>
@tbarsballe
Copy link
Contributor

tbarsballe commented Apr 7, 2020

Edit: Comment was on the wrong PR, see geoserver/geoserver#4165

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.

None yet

2 participants