[pacman-dev] [PATCH v2] makepkg --packagelist: just list the built package files we will build
allan at archlinux.org
Thu Feb 8 04:10:20 UTC 2018
On 08/02/18 13:44, Eli Schwartz wrote:
> On 02/07/2018 10:37 PM, Allan McRae wrote:
>> On 08/02/18 11:22, Eli Schwartz wrote:
>>> On 02/07/2018 07:24 PM, Allan McRae wrote:
>>>> Newline separation should be fine. makepkg ensures most (all?) of the
>>>> fields are not allowed to have newlines in them. If any can, we need to
>>>> catch that in out PKGBUILD linting and error out. Probably need to add
>>>> a check that the fields from makepkg.conf don't have newlines either.
>>> I was thinking primarily of the fields in makepkg.conf, since AFAICT no
>>> PKGBUILD fields used in assembling the filename can contain newlines.
>>> OTOH, both PKGDEST and PKGEXT currently allow this, as do all the other
>>> directory variables in makepkg.conf
>>> Should we add a lint_pkgbuild check for all the makepkg.conf variables?
>> Yes please. Although, lint_config.sh might be a better place to put
>> it, as it is not really part of pkgbuild linting.
> Sure, anything else that we should lint while we are at it? I think
> mostly the only things to check are variable types, which we could
> probably crib from lint_pkgbuild/variable.sh
We should only lint things that we want to ensure match assumptions.
For this patch, checking CARCH and PKGEXT are strings and don't contain
newlines is enough.
More information about the pacman-dev