Skip to content

CI: Switch to OSGeo4W V2 - #2074

Merged
ninsbl merged 4 commits into
OSGeo:mainfrom
ninsbl:osgeo4w_v2_ci
Jan 11, 2022
Merged

ninsbl merged 4 commits into
OSGeo:mainfrom
ninsbl:osgeo4w_v2_ci

Conversation

@ninsbl

@ninsbl ninsbl commented Jan 7, 2022

Copy link
Copy Markdown
Member

This brings back CI on MS Windows. It replaces #1873

@HuidaeCho do you have any objections to merge this now, and adress packaging in another PR?

@ninsbl ninsbl added windows Microsoft Windows specific CI Continuous integration labels Jan 7, 2022
@ninsbl ninsbl added this to the 8.0.0 milestone Jan 7, 2022
@ninsbl
ninsbl requested review from HuidaeCho and wenzeslaus January 7, 2022 14:47
@hellik

hellik commented Jan 7, 2022

Copy link
Copy Markdown
Member

thanks for bring back windows CI.

PR looks good.

@neteler

neteler commented Jan 10, 2022

Copy link
Copy Markdown
Member

@ninsbl May this be merged? Would be nice to see the CI completely running again!

@hellik

hellik commented Jan 10, 2022

Copy link
Copy Markdown
Member

@ninsbl May this be merged? Would be nice to see the CI completely running again!

would be good to have win CI back before 8.0 release.

@ninsbl

ninsbl commented Jan 10, 2022

Copy link
Copy Markdown
Member Author

@ninsbl May this be merged? Would be nice to see the CI completely running again!

From my perspective, yes.
We can fix the packaging issue with a separate (new) script in the mswindows directory, what do you think @HuidaeCho ...?

@neteler

neteler commented Jan 11, 2022

Copy link
Copy Markdown
Member

Please let's go ahead and merge. We need the Windows CI back.

@ninsbl
ninsbl merged commit 53dfd4a into OSGeo:main Jan 11, 2022
@ninsbl

ninsbl commented Jan 11, 2022

Copy link
Copy Markdown
Member Author

Should we backport to 8.0 releasebranch? It does not (should not) affect the sofyware / packages delivered to endusers, so it should be safe...

neteler pushed a commit that referenced this pull request Jan 11, 2022
With this PR merged, CI uses packages from OSGeo4W Version2 (https://github.com/jef-n/OSGeo4W/) to build GRASS GIS using MSYS. Now also shellscript tests are executed. The PR replaces #1873 and parts of #1683. Compilation for tests is now also configured with OpenMP and LAPACK support (amongst others).
@neteler

neteler commented Jan 11, 2022

Copy link
Copy Markdown
Member

Should we backport to 8.0 releasebranch? It does not (should not) affect the sofyware / packages delivered to endusers, so it should be safe...

Yes - I have just done that.

@wenzeslaus wenzeslaus changed the title CI for MS Windows: Switch to OSGeo4W V2 CI: Switch to OSGeo4W V2 Apr 27, 2022
ninsbl added a commit to ninsbl/grass that referenced this pull request Oct 26, 2022
With this PR merged, CI uses packages from OSGeo4W Version2 (https://github.com/jef-n/OSGeo4W/) to build GRASS GIS using MSYS. Now also shellscript tests are executed. The PR replaces OSGeo#1873 and parts of OSGeo#1683. Compilation for tests is now also configured with OpenMP and LAPACK support (amongst others).
ninsbl added a commit to ninsbl/grass that referenced this pull request Feb 17, 2023
With this PR merged, CI uses packages from OSGeo4W Version2 (https://github.com/jef-n/OSGeo4W/) to build GRASS GIS using MSYS. Now also shellscript tests are executed. The PR replaces OSGeo#1873 and parts of OSGeo#1683. Compilation for tests is now also configured with OpenMP and LAPACK support (amongst others).
neteler pushed a commit to nilason/grass that referenced this pull request Nov 7, 2023
With this PR merged, CI uses packages from OSGeo4W Version2 (https://github.com/jef-n/OSGeo4W/) to build GRASS GIS using MSYS. Now also shellscript tests are executed. The PR replaces OSGeo#1873 and parts of OSGeo#1683. Compilation for tests is now also configured with OpenMP and LAPACK support (amongst others).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous integration windows Microsoft Windows specific

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants