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

Wrong paths on Arch Linux / AUR #10

Closed
plntyk opened this issue Aug 24, 2018 · 7 comments
Closed

Wrong paths on Arch Linux / AUR #10

plntyk opened this issue Aug 24, 2018 · 7 comments

Comments

@plntyk
Copy link

plntyk commented Aug 24, 2018

Installing on Arch Linux via AUR (kmscon-patched-git, libtsm-patched-git) now does not work anymore
after latest change #9

running kmscon does not work because /usr/bin/kmscon contains the wrong path

AUR package creates a

/usr/lib/kmscon/kmscon/kmscon

Editing AUR configure options and/or kmscon shellscript that calls the binary manually fixed the situation so that for example systemctl start kmscon works again

@PegasusEpsilon
Copy link

PegasusEpsilon commented Sep 5, 2018

Can confirm, and it's not just from those PKGBUILDs:

pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ grep url .git/config
	url = https://github.com/Aetf/kmscon.git
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ git reset --hard
HEAD is now at 188a960 Fix file conflict during instal, close #9
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ make clean distclean
...elided...
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ git status
On branch master
Your branch is up to date with 'origin/master'.

nothing to commit, working tree clean
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ git pull
Already up to date.
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ ls
build-aux/  external/  m4/       src/    autom4te.cache/  COPYING  README        Makefile.am  Makefile.in  autogen.sh*
docs/       fblog/     scripts/  tests/  configure*       NEWS     configure.ac  config.h.in  aclocal.m4
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ ./configure
...elided...
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ make
...elided...
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ make install DESTDIR=~/.trizen/arch-kmscon-custom/pkg
...elided...
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ tail -n1 ../pkg/usr/local/bin/kmscon 
exec ${libexecdir}/kmscon "$@"
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ grep libexecdir ../pkg/usr/local/bin/kmscon | head -n1
libexecdir=${exec_prefix}/libexec
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ grep exec_prefix ../pkg/usr/local/bin/kmscon | head -n1
exec_prefix=${prefix}
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ grep prefix ../pkg/usr/local/bin/kmscon | head -n1
prefix=/usr/local
pegasus@Li⁺:~/.trizen/arch-kmscon-custom/src$ file ../pkg/usr/local/libexec/kmscon
../pkg/usr/local/libexec/kmscon: directory

This was caused by 188a960 -- the commit before it (3f8b688) works just fine. It looks to me like the patch was pushed in without including a modified scripts/kmscon.

@Aetf it should be a simple fix, if your local version works. I can't imagine this is working for anyone but you, right now.

@Aetf
Copy link
Owner

Aetf commented Sep 5, 2018

Sorry for the careless fix :P

I've updated both the PKGBUILD and scripts/kmscon to be consistent with the path change.

Could both of you @plntyk @PegasusEpsilon test before I close this?

@LChris314
Copy link

Not those two users but I've just updated and it worked fine. File list shown by pacman -Ql shows correct path.

@plntyk
Copy link
Author

plntyk commented Sep 6, 2018

Uninstalled and reinstalled kmscon-patched-git via AUR and it works.

@Aetf
Copy link
Owner

Aetf commented Sep 6, 2018

Okay. I'm closing this then. Fixed in d2c0134

@Aetf Aetf closed this as completed Sep 6, 2018
@PegasusEpsilon
Copy link

Sorry for the delay, can confirm that this has been fixed. Thanks, @Aetf!

@Aetf
Copy link
Owner

Aetf commented Sep 6, 2018

@PegasusEpsilon Cool!

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

No branches or pull requests

4 participants