Re: [packages/FHS] introduce /usr/{,local/}libexec directories

2017-07-10 Wątek Jan Rękorajski
On Mon, 10 Jul 2017, Jakub Bogusz wrote:

> On Mon, Jul 10, 2017 at 04:57:30PM +0200, Tomasz Pala wrote:
> > On Mon, Jul 10, 2017 at 20:02:48 +0900, Jan Rękorajski wrote:
> > 
> > > If you want me to keep this commit and directory then follow up by:
> > > 
> > > a) updating rpm macros
> > 
> > Yes, I was considering this point. Just wondering, what would break (in
> > theory: nothing should) and how to perform the validation. Didn't want
> > to do such change without more feedback, so now - if you already
> > summoned this subject, I'll wait a few days for any comments.
> > 
> > I've already reviewed these and only one (re)definition needs to be
> > adjusted (in /usr/lib/rpm/macros.d/pld), remaining macros seem to be
> > cascading properly.
> 
> Note that there are some inter-package consistency requirements.
> 
> And just like some packages having hardcoded /usr/libexec, and "require
> hackery" to use libdir subdirectory, the others have hardcoded /usr/lib**
> for this purpose and would "require hackery" to use libexec.
> 
> Without using libexec consequently, I don't see any profits (single
> place for internal binaries).

I see a profit - not doing that hackery. The other hackery is for mixing
arch and noarch subpackages. Right now it's either all or nothing -
libdir or datadir (see git-core ping-pong), or hacking program to understand
both locations. With libexec dir we'll have it the way author wanted and
will be able to build noarch subpackages.

-- 
Jan Rękorajski| PLD/Linux
SysAdm | bagginspld-linux.org | http://www.pld-linux.org/
___
pld-devel-pl mailing list
pld-devel-pl@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-pl


Re: [packages/FHS] introduce /usr/{,local/}libexec directories

2017-07-10 Wątek Jakub Bogusz
On Mon, Jul 10, 2017 at 04:57:30PM +0200, Tomasz Pala wrote:
> On Mon, Jul 10, 2017 at 20:02:48 +0900, Jan Rękorajski wrote:
> 
> > If you want me to keep this commit and directory then follow up by:
> > 
> > a) updating rpm macros
> 
> Yes, I was considering this point. Just wondering, what would break (in
> theory: nothing should) and how to perform the validation. Didn't want
> to do such change without more feedback, so now - if you already
> summoned this subject, I'll wait a few days for any comments.
> 
> I've already reviewed these and only one (re)definition needs to be
> adjusted (in /usr/lib/rpm/macros.d/pld), remaining macros seem to be
> cascading properly.

Note that there are some inter-package consistency requirements.

And just like some packages having hardcoded /usr/libexec, and "require
hackery" to use libdir subdirectory, the others have hardcoded /usr/lib**
for this purpose and would "require hackery" to use libexec.

Without using libexec consequently, I don't see any profits (single
place for internal binaries).

> > b) cleaning up packages that have libexec redefined directly in specs
> 
> > FHS states this directory is optional, and I do not care at all what GNU
> > shamans think. This is not GNU/PLD, just PLD.
> 
> I don't care about all this GNU/crap either, but using some Fedora
> systems this directory was really convenient. My personal rationale is
> 'follow the world', just to avoid being different than all the rest.

On the other side, the "second half of the world" (Debian/Ubuntu) doesn't
use libexec.
>From minorities, e.g. Gentoo uses, Arch doesn't.


-- 
Jakub Boguszhttp://qboosh.pl/
___
pld-devel-pl mailing list
pld-devel-pl@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-pl


Re: [packages/FHS] introduce /usr/{,local/}libexec directories

2017-07-10 Wątek Tomasz Pala
On Mon, Jul 10, 2017 at 20:02:48 +0900, Jan Rękorajski wrote:

> If you want me to keep this commit and directory then follow up by:
> 
> a) updating rpm macros

Yes, I was considering this point. Just wondering, what would break (in
theory: nothing should) and how to perform the validation. Didn't want
to do such change without more feedback, so now - if you already
summoned this subject, I'll wait a few days for any comments.

I've already reviewed these and only one (re)definition needs to be
adjusted (in /usr/lib/rpm/macros.d/pld), remaining macros seem to be
cascading properly.

> b) cleaning up packages that have libexec redefined directly in specs

> FHS states this directory is optional, and I do not care at all what GNU
> shamans think. This is not GNU/PLD, just PLD.

I don't care about all this GNU/crap either, but using some Fedora
systems this directory was really convenient. My personal rationale is
'follow the world', just to avoid being different than all the rest.

-- 
Tomasz Pala 
___
pld-devel-pl mailing list
pld-devel-pl@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-pl


Re: [packages/FHS] introduce /usr/{,local/}libexec directories

2017-07-10 Wątek Jan Rękorajski
If you want me to keep this commit and directory then follow up by:

a) updating rpm macros
b) cleaning up packages that have libexec redefined directly in specs

FHS states this directory is optional, and I do not care at all what GNU
shamans think. This is not GNU/PLD, just PLD.

On Thu, 06 Jul 2017, gotar wrote:

> commit d49d853dc96ef5cd1a7d7f8f900577608a38be03
> Author: Tomasz Pala 
> Date:   Thu Jul 6 11:41:42 2017 +0200
> 
> introduce /usr/{,local/}libexec directories
> 
> https://refspecs.linuxfoundation.org/FHS_3.0/fhs/ch04s07.html
> 
> Note, that /usr/local/libexec is not mentioned explicitly in
> https://refspecs.linuxfoundation.org/FHS_3.0/fhs-3.0.html#requirements10,
> but rationale behind both libexec and local hierarchy allow us to treat
> this as lib.
> 
> Since /usr/local/libexec should exist for GNU applications according to
> https://www.gnu.org/prep/standards/html_node/Directory-Variables.html,
> we must provide /usr/libexec directory as well to met FHS requirements.
> 
>  FHS.spec | 8 +---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> ---
> diff --git a/FHS.spec b/FHS.spec
> index f92f737..3291fdd 100644
> --- a/FHS.spec
> +++ b/FHS.spec
> @@ -12,7 +12,7 @@ Summary(pl.UTF-8):  Podstawowy układ katalogów systemu 
> Linux zgodny z FHS 3.0
>  Summary(tr.UTF-8):   Temel dosya sistemi yapısı
>  Name:FHS
>  Version: 3.0
> -Release: 1
> +Release: 2
>  License: GPL
>  Group:   Base
>  URL: http://refspecs.linuxfoundation.org/fhs.shtml
> @@ -83,10 +83,10 @@ install -d \
>   $RPM_BUILD_ROOT/etc/{X11,opt} \
>   $RPM_BUILD_ROOT/lib/modules \
>   $RPM_BUILD_ROOT/{mnt,media,proc,root/tmp,sbin,tmp} \
> - $RPM_BUILD_ROOT/usr/{bin,games,include,lib,sbin,share,src} \
> + $RPM_BUILD_ROOT/usr/{bin,games,include,lib{,exec},sbin,share,src} \
>   
> $RPM_BUILD_ROOT/usr/share/{color/icc,dict,doc,games,info,misc,ppd,tmac,xml} \
>   $RPM_BUILD_ROOT/usr/lib/games \
> - 
> $RPM_BUILD_ROOT/usr/local/{bin,etc,games,include,lib,sbin,share/{color/icc,doc,info,man},src}
>  \
> + 
> $RPM_BUILD_ROOT/usr/local/{bin,etc,games,include,lib{,exec},sbin,share/{color/icc,doc,info,man},src}
>  \
>   
> $RPM_BUILD_ROOT/var/{cache,crash,db,games,lib/{color/icc,misc},local,lock,log,mail,opt,run,spool,tmp,yp}
>  
>  %if %{with lib64}
> @@ -165,6 +165,7 @@ posix.chown("/var/lock", 0, %{gid_uucp})
>  %dir /usr/games
>  %dir /usr/include
>  %dir /usr/lib
> +%dir /usr/libexec
>  %dir /usr/lib/games
>  %dir /usr/sbin
>  %dir /usr/share
> @@ -187,6 +188,7 @@ posix.chown("/var/lock", 0, %{gid_uucp})
>  %dir /usr/local/games
>  %dir /usr/local/include
>  %dir /usr/local/lib
> +%dir /usr/local/libexec
>  %dir /usr/local/sbin
>  %dir /usr/local/share
>  %dir /usr/local/share/color
> 
> 
>  gitweb:
> 
> http://git.pld-linux.org/gitweb.cgi/packages/FHS.git/commitdiff/d49d853dc96ef5cd1a7d7f8f900577608a38be03
> 
> ___
> pld-cvs-commit mailing list
> pld-cvs-com...@lists.pld-linux.org
> http://lists.pld-linux.org/mailman/listinfo/pld-cvs-commit

-- 
Jan Rękorajski| PLD/Linux
SysAdm | bagginspld-linux.org | http://www.pld-linux.org/
___
pld-devel-pl mailing list
pld-devel-pl@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-pl