Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2020-03-04 Thread Florian Festi
Closed #714. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/714#event-3096849270___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2020-03-04 Thread Florian Festi
Moved to rpmlint as https://github.com/rpm-software-management/rpmlint/issues/396 -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2019-07-14 Thread ニール・ゴンパ
@jasontibbitts In Mageia and OpenMandriva, rpmlint is executed as a post-build check through rpmbuild itself, like so: ```rpmspec %_build_pkgcheck_set /usr/bin/rpmlint -f %{_sourcedir}/%{name}.rpmlintrc %_build_pkgcheck_srpm /usr/bin/rpmlint -f %{_sourcedir}/%{name}.rpmlintrc

Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2019-07-11 Thread Jason Tibbitts
This would probably be better as an rpmlint check, except then it might be ignored. Certainly from RPM's perspective, having a %pretrans scriptlet written in shell or anything else is perfectly valid. Only a distro like Fedora which cares about every package being installable into an empty

Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2019-05-28 Thread Panu Matilainen
Nope, still works as expected... ``` [root@sopuli Packages]# rpm --root /srv/test -Uvh b/bash-4.4.19-2.fc28.x86_64.rpm g/glibc-2.27-8.fc28.x86_64.rpm g/glibc-common-2.27-8.fc28.x86_64.rpm n/ncurses-libs-6.1-4.20180224.fc28.x86_64.rpm n/ncurses-base-6.1-4.20180224.fc28.noarch.rpm

Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2019-05-28 Thread Panu Matilainen
Yes it will, that's the whole point. If it doesn't, then we've broken something. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2019-05-28 Thread Florian Festi
Well, but it will not fail if you install the bash package with it. Still the pretrans scriptlet won't work. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2019-05-28 Thread Panu Matilainen
> This issue is kinda worsened by the fact that this will not create a > dependency issue but rather an ordering problem that is more or less ignored > up to the point where the script fails. Um, what do you mean? It *is* a dependency issue, a package with /bin/sh (or similar) %pretrans

Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2019-05-28 Thread Florian Festi
Well, we should at least issue a warning that the package cannot be installed on an empty system. This issue is kinda worsened by the fact that this will not create a dependency issue but rather an ordering problem that is more or less ignored up to the point where the script fails. -- You

Re: [Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2019-05-27 Thread Panu Matilainen
Well, technically it's perfectly okay to have a package that's only installable after the OS installation, not all packages live in repositories and get installed as a part of system installation. We also do handle this correctly for almost a decade - such a package fails cleanly

[Rpm-maint] [rpm-software-management/rpm] Reject %pretrans scripts that are not lua (#714)

2019-05-26 Thread Florian Festi
There is no way to run a %pretrans script that relies on anyhting but lua as it will run on an empty system for new installs. RPM should just reject to create such scriptlet in the first place. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or