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

OsGeo4W V2 update #2

Open
nicogodet opened this issue Oct 2, 2021 · 2 comments · May be fixed by #3
Open

OsGeo4W V2 update #2

nicogodet opened this issue Oct 2, 2021 · 2 comments · May be fixed by #3
Labels
enhancement New feature or request

Comments

@nicogodet
Copy link

I started working on the update of your awesome work.

Few typos and some changes in postinstall and preremove script (avoid the use of nircmd).
Still WIP but I will isolate the main change and propose a PR, I hope we can both work on this.

@haubourg haubourg added the enhancement New feature or request label Oct 2, 2021
@haubourg
Copy link
Owner

haubourg commented Oct 2, 2021

Wow that's awesome, thanks a lot @nicogodet !

A Pull request will be very welcome when you feel ready.
At first glance, you will have to replace all 'isl' occurences back to 'custom' to keep it generic.
I'll probably work on this for my new employer again, most probably in 2022, but I'll be happy to test.
Also feel free to talk about this on qgis-developper and osgeo4W lists and bug trackers if you find upstream issues or enhancement.

poke @Utopiageo this can help you upgrading faster to next OSGEO4W installer generation

@nicogodet
Copy link
Author

Of course, the isl branche will not be used for PR. I will create another branch and pick some fixes from isl.

Currently, install works like a charm with V2, I still have issue with desktop shortcut. OSGEO4W_DESKTOP_LINKS equals 0 even if I don't specify --no-desktop.
I need to figure out how to create a oneline command for upgrades.

@nicogodet nicogodet linked a pull request Oct 2, 2021 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants