datalad / datalad-gooey

A graphical user interface for DataLad (datalad.org)
https://docs.datalad.org/projects/gooey
Other
4 stars 6 forks source link

Add a gooey-only installer that can be installed without admin privileges #430

Closed christian-monch closed 10 months ago

christian-monch commented 10 months ago

This PR implements option 1 of issue #425. It modifies the "Build Windows Installer action to build a second installer that will only install datalad-gooey, datalad, and python 3.9 in user directories, i.e. <user>\AppData\Local\datalad.org.

codecov-commenter commented 10 months ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.07% :tada:

Comparison is base (6c218f6) 60.53% compared to head (7aab36e) 60.60%. Report is 16 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #430 +/- ## ========================================== + Coverage 60.53% 60.60% +0.07% ========================================== Files 49 49 Lines 3512 3511 -1 ========================================== + Hits 2126 2128 +2 + Misses 1386 1383 -3 ``` | [Files Changed](https://app.codecov.io/gh/datalad/datalad-gooey/pull/430?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad) | Coverage Δ | | |---|---|---| | [datalad\_gooey/constraints.py](https://app.codecov.io/gh/datalad/datalad-gooey/pull/430?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9nb29leS9jb25zdHJhaW50cy5weQ==) | `83.47% <100.00%> (ø)` | | | [datalad\_gooey/credentials.py](https://app.codecov.io/gh/datalad/datalad-gooey/pull/430?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9nb29leS9jcmVkZW50aWFscy5weQ==) | `12.50% <100.00%> (ø)` | | | [datalad\_gooey/patches/clone.py](https://app.codecov.io/gh/datalad/datalad-gooey/pull/430?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9nb29leS9wYXRjaGVzL2Nsb25lLnB5) | `77.77% <100.00%> (-2.23%)` | :arrow_down: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/datalad/datalad-gooey/pull/430/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

christian-monch commented 10 months ago

ah wait, one second - something is weird. I did not notice the first time I ran the full installer, but when I redid it after wiping everything just for good measure, I noticed that the full installer does not seem to install git-annex anymore? I can't pinpoint where in this PR this would happen, but executing the full installer on an admin account does not prompt for the admin password, and does not install git-annex.

Thanks for the report. Checking this now

christian-monch commented 10 months ago

Thanks a lot @adswa

Thanks @christian-monch. I left two small wording suggestions for the docs. I tested both installers on Windows 10, and they seemed to work as advertised - yay! I only have two general remarks, probably for a follow-up PR: The overview of releases currently looks like this

image

I suspect that the Git-SHA-based name looks more weird to many users than the old installer with the "clean" v0.2.0. Should we do a patch release in order to have those two most recent installers readily and easily identifiable?

Yes, will do that

I noticed when using the full installer that Git 2.37.3 was being installed, this is 5 minor versions below the current version and was released more than one year ago. Its likely useful to update this.

Good point. Created issue https://github.com/datalad/datalad-gooey/issues/434 and assigned it to myself.

If users already have a more recent git version they would otherwise be forced to downgrade. Thanks for noticing that. Users might indeed downgrade, but they are not forced to do that. Users are free to cancel the git installation. I added a note about this topic to the installation instructions for windows

christian-monch commented 10 months ago

ah wait, one second - something is weird. I did not notice the first time I ran the full installer, but when I redid it after wiping everything just for good measure, I noticed that the full installer does not seem to install git-annex anymore? I can't pinpoint where in this PR this would happen, but executing the full installer on an admin account does not prompt for the admin password, and does not install git-annex.

Found it. The git-annex installer, which requests admin privileges, is only started by an installer that has requested admin privileges. Long story short, I fixed it by requesting admin level for the full installer (in 7aab36e)