fsquillace / junest

The lightweight Arch Linux based distro that runs, without root privileges, on top of any other Linux distro.
GNU General Public License v3.0
2.1k stars 110 forks source link

Junest setup fails when using JUNEST_HOME env var #335

Closed adworacz closed 1 year ago

adworacz commented 1 year ago

I'm not sure that the issue has anything to do with the env var usage, but it seems like the .junest/usr/bin directory is not being created properly upon setup.

Here's an example of trying to install in a fresh directory:

╰─  JUNEST_HOME=./.junest junest setup
Downloading JuNest...
--2023-07-11 21:12:56--  https://pub-a2af2344e8554f6c807bc3db355ae622.r2.dev/junest/junest-x86_64.tar.gz
Loaded CA certificate '/etc/ssl/certs/ca-certificates.crt'
Resolving pub-a2af2344e8554f6c807bc3db355ae622.r2.dev (pub-a2af2344e8554f6c807bc3db355ae622.r2.dev)... 104.18.3.35
Connecting to pub-a2af2344e8554f6c807bc3db355ae622.r2.dev (pub-a2af2344e8554f6c807bc3db355ae622.r2.dev)|104.18.3.35|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 191058952 (182M) [application/x-tar]
Saving to: ‘junest-x86_64.tar.gz’

junest-x86_64.tar.gz    100%[============================>] 182.21M  5.72MB/s    in 48s     

2023-07-11 21:13:46 (3.76 MB/s) - ‘junest-x86_64.tar.gz’ saved [191058952/191058952]

Installing JuNest...
JuNest installed successfully!

Default mirror URL set to: https://mirror.rackspace.com/archlinux/$repo/os/$arch
You can change the pacman mirror URL in /etc/pacman.d/mirrorlist according to your location:
    $EDITOR ./.junest/etc/pacman.d/mirrorlist

Remember to refresh the package databases from the server:
    pacman -Syy

To install packages from AUR follow the wiki here:
https://github.com/fsquillace/junest#install-packages-from-aur
/opt/junest/lib/core/wrappers.sh: line 30: ./.junest/usr/bin/junest_wrapper: No such file or directory
fsquillace commented 1 year ago

If I remember correctly JUNEST_HOME works only with absolute paths.

adworacz commented 1 year ago

Got it - using JUNEST_HOME="$(pwd)/.junest" junest setup worked like a charm.

Might I request that the documentation be updated to indicate the absolute path requirement?

Other than that, I think this issue can be closed.

fsquillace commented 1 year ago

Makes sense. Merged the change in dev https://github.com/fsquillace/junest/commit/6ae9f72dd5bd423bf5e6db8e62cf0e343a8a2bee