Re: [Rpm-maint] [rpm-software-management/rpm] Add support for generating buildinfo file as subpackage (#1532)
Thank you all for your feedback, I'll prepare another iteration soon with all the comments here. -- 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/pull/1532#issuecomment-943021678___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Add support for generating buildinfo file as subpackage (#1532)
@fepitre commented on this pull request. > + +printf 'Installed-Build-Depends:\n' >> "$BUILDINFO" +rpm -qa --queryformat '%{epoch}:%{name}-%{version}-%{release}.%{arch}\n' \ +| LC_ALL=C sort -t: -k2 \ +| sed -e 's/^(none)://; /\.(none)$/d; s/^/ /' >> "$BUILDINFO" + +printf 'Environment:\n' >> "$BUILDINFO" + +# Whitelist from Debian's Dpkg: +# https://anonscm.debian.org/git/dpkg/dpkg.git/tree/scripts/Dpkg/Build/Info.pm#n50 +ENV_WHITELIST= + +# Toolchain. +ENV_WHITELIST="$ENV_WHITELIST CC CPP CXX OBJC OBJCXX PC FC M2C AS LD AR RANLIB MAKE AWK LEX YACC" +# Toolchain flags. +ENV_WHITELIST="$ENV_WHITELIST CFLAGS CPPFLAGS CXXFLAGS OBJCFLAGS OBJCXXFLAGS GCJFLAGS FFLAGS LDFLAGS ARFLAGS MAKEFLAGS" Ah I see. Thank you for the enlightenment. -- 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/pull/1532#discussion_r728673531___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Add support for generating buildinfo file as subpackage (#1532)
@fepitre commented on this pull request. > + +mkdir -p "$BUILDINFO_DIR" + +cat > "$BUILDINFO" <> "$BUILDINFO" @Conan-Kudo thank you for your feedback and comments. Generally I would have loved to have a generic format not being one distro specific to ease manipulating this file among several rebuild tools but I guess it would not be straightforward. I'm waiting some feedback from @bmwiedemann too then I would propose to adapt the work according to your comments. -- 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/pull/1532#discussion_r728399795___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Add support for generating buildinfo file as subpackage (#1532)
@bmwiedemann do you have some feedback to give from several comments here? As you work on reproducible builds for openSUSE you are certainly interested by this new feature. -- 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/pull/1532#issuecomment-942664681___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Add support for generating buildinfo file as subpackage (#1532)
@fepitre commented on this pull request. > + +printf 'Installed-Build-Depends:\n' >> "$BUILDINFO" +rpm -qa --queryformat '%{epoch}:%{name}-%{version}-%{release}.%{arch}\n' \ +| LC_ALL=C sort -t: -k2 \ +| sed -e 's/^(none)://; /\.(none)$/d; s/^/ /' >> "$BUILDINFO" + +printf 'Environment:\n' >> "$BUILDINFO" + +# Whitelist from Debian's Dpkg: +# https://anonscm.debian.org/git/dpkg/dpkg.git/tree/scripts/Dpkg/Build/Info.pm#n50 +ENV_WHITELIST= + +# Toolchain. +ENV_WHITELIST="$ENV_WHITELIST CC CPP CXX OBJC OBJCXX PC FC M2C AS LD AR RANLIB MAKE AWK LEX YACC" +# Toolchain flags. +ENV_WHITELIST="$ENV_WHITELIST CFLAGS CPPFLAGS CXXFLAGS OBJCFLAGS OBJCXXFLAGS GCJFLAGS FFLAGS LDFLAGS ARFLAGS MAKEFLAGS" I don't understand your point. This is not to set with %configure. It's the job of rebuilder like https://github.com/fepitre/rpmreproduce -- 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/pull/1532#discussion_r728396243___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Add support for generating buildinfo file as subpackage (#1532)
> So start by explaining what problem you're trying to solve, and not how other > people might've chosen to solve it. The purpose of having an automatically generated subpackage `buildinfo` is to have an exact description of the build environment used to obtain RPMs from a spec file. Having this, we can rebuild in the same condition any `src.rpm` (see my WIP tool called [rpmreproduce](https://github.com/fepitre/rpmreproduce/tree/devel-1.0-rpm) and compare with the original RPMs. Also, if a given package is supposed to be reproducible and rebuild differs, something is wrong and needs to be investigated. -- 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/pull/1532#issuecomment-809310265___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Add support for generating buildinfo file as subpackage (#1532)
Just a little ping here to know if anyone has already take a look on the global form for this. @Conan-Kudo any feedback if you have time? Thank you very much. -- 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/pull/1532#issuecomment-782911313___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
[Rpm-maint] [rpm-software-management/rpm] Add support for generating buildinfo file as subpackage (#1532)
TODO: - Add Binary field with all built RPMs package names - Add Checksums for at least source RPM - Add additional rpm macros (--define, --with/--without, etc.) Reference: https://wiki.debian.org/ReproducibleBuilds/BuildinfoFiles, Thread Reproducible builds on Fedora devel list. You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/1532 -- Commit Summary -- * Add support for generating buildinfo file as subpackage -- File Changes -- M macros.in (28) A scripts/rpmbuildinfo (96) -- Patch Links -- https://github.com/rpm-software-management/rpm/pull/1532.patch https://github.com/rpm-software-management/rpm/pull/1532.diff -- 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/pull/1532 ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint