Re: [gentoo-dev] [PATCH v3 1/1] greadme.eclass: new eclass

2024-06-13 Thread Florian Schmaus
On 13/06/2024 11.31, Ulrich Mueller wrote: On Thu, 13 Jun 2024, Florian Schmaus wrote: +# Copyright 1999-2024 Gentoo Authors +# Distributed under the terms of the GNU General Public License v2 + +# @ECLASS: greadme.eclass +# @MAINTAINER: +# Florian Schmaus +# @AUTHOR: +# Author: Florian Schmaus

Re: [gentoo-dev] [PATCH v3 1/1] greadme.eclass: new eclass

2024-06-13 Thread Ulrich Mueller
> On Thu, 13 Jun 2024, Ulrich Mueller wrote: >> +if $append; then > Please use the conventional coding style, i.e. ${append}. Or rather, assign the variable to empty or non-empty and test for [[ -n ${append} ]] instead of executing it as a true or false command. (I'm pretty sure that th

Re: [gentoo-dev] [PATCH v3 1/1] greadme.eclass: new eclass

2024-06-13 Thread Florian Schmaus
On 13/06/2024 12.42, Ulrich Mueller wrote: On Thu, 13 Jun 2024, Florian Schmaus wrote: +_GREADME_DOC_DIR="usr/share/doc/${PF}" It is somewhat unusual to call insinto or docompress with a relative path. I'd use "/usr/share/doc/${PF}" here. +_GREADME_REL_PATH="${_GREADME_DOC_DIR}/${_GREADME_FIL

Re: [gentoo-dev] [PATCH v3 1/1] greadme.eclass: new eclass

2024-06-13 Thread Ulrich Mueller
> On Thu, 13 Jun 2024, Florian Schmaus wrote: >>> +_GREADME_DOC_DIR="usr/share/doc/${PF}" >> It is somewhat unusual to call insinto or docompress with a relative >> path. I'd use "/usr/share/doc/${PF}" here. >> >>> +_GREADME_REL_PATH="${_GREADME_DOC_DIR}/${_GREADME_FILENAME}" >> Why must this

Re: [gentoo-dev] [PATCH v3 1/1] greadme.eclass: new eclass

2024-06-13 Thread Ulrich Mueller
> On Thu, 13 Jun 2024, Florian Schmaus wrote: > +# Copyright 1999-2024 Gentoo Authors > +# Distributed under the terms of the GNU General Public License v2 > + > +# @ECLASS: greadme.eclass > +# @MAINTAINER: > +# Florian Schmaus > +# @AUTHOR: > +# Author: Florian Schmaus > +# @SUPPORTED_EAPIS

[gentoo-dev] [PATCH v3 1/1] greadme.eclass: new eclass

2024-06-13 Thread Florian Schmaus
This new eclass includes various improvements over the existing readme.gentoo-r1.eclass. First, the content of README.gento will only be shown on new installations or if it has changed between updates. Second, it allows the composition of readme via bash's heredoc. Third, it exports phase functi