On Mon, 03 Jul 2017, Jeffrey Johnson wrote: > > > On Jul 3, 2017, at 2:07 AM, Jan Rękorajski <[email protected]> wrote: > > > > Why do you keep adding that cruft? Please stop doing this. > > We don't use rpm4 for 5 (five) years. > > > > Hmmm … there’s the other incompatibility side as well > > Why isn’t BuildArch needed for RPM4? > > I can remove (or at least make optional) the need for "BuildArch: noarch" in > RPM5 if necessary. > > (aside) > The BuildArch: directive forces rpm build to abort the current build, > discard/reread entire configuration, > and restart build parsing. SO its one if the crazier pieces of code in RPM, > always has been. > > But if rpm.org has removed the beed for BuildArch: no arch, I’m sure I can do > the same in RPM5.
You don't need to remove anything. This directive here is for antiquated rpm4 that did not support BuildArch in subpackages. I want those %if's gone. > > 73 de Jeff > > > On Sun, 02 Jul 2017, glen wrote: > > > >> commit b19b9d5139395caf6414506d1baa84d8bc1b69e6 > >> Author: Elan Ruusamäe <[email protected]> > >> Date: Sun Jul 2 23:46:57 2017 +0300 > >> > >> rpm4 noarch > >> > >> rust.spec | 8 ++++++++ > >> 1 file changed, 8 insertions(+) > >> --- > >> diff --git a/rust.spec b/rust.spec > >> index 7e7d09a..91be41a 100644 > >> --- a/rust.spec > >> +++ b/rust.spec > >> @@ -95,7 +95,9 @@ documentation generator. > >> %package debugger-common > >> Summary: Common debugger pretty printers for Rust > >> Group: Development/Debuggers > >> +%if "%{_rpmversion}" >= "5" > >> BuildArch: noarch > >> +%endif > >> > >> %description debugger-common > >> This package includes the common functionality for %{name}-gdb and > >> @@ -106,7 +108,9 @@ Summary: GDB pretty printers for Rust > >> Group: Development/Debuggers > >> Requires: %{name}-debugger-common = %{version}-%{release} > >> Requires: gdb > >> +%if "%{_rpmversion}" >= "5" > >> BuildArch: noarch > >> +%endif > >> > >> %description gdb > >> This package includes the rust-gdb script, which allows easier > >> @@ -117,7 +121,9 @@ Summary: LLDB pretty printers for Rust > >> Group: Development/Debuggers > >> Requires: %{name}-debugger-common = %{version}-%{release} > >> Requires: lldb > >> +%if "%{_rpmversion}" >= "5" > >> BuildArch: noarch > >> +%endif > >> > >> %description lldb > >> This package includes the rust-lldb script, which allows easier > >> @@ -126,7 +132,9 @@ debugging of Rust programs. > >> %package doc > >> Summary: Documentation for Rust > >> Group: Documentation > >> +%if "%{_rpmversion}" >= "5" > >> BuildArch: noarch > >> +%endif > >> > >> %description doc > >> This package includes HTML documentation for the Rust programming > >> ================================================================ > >> > >> ---- gitweb: > >> > >> http://git.pld-linux.org/gitweb.cgi/packages/rust.git/commitdiff/b19b9d5139395caf6414506d1baa84d8bc1b69e6 > >> > >> _______________________________________________ > >> pld-cvs-commit mailing list > >> [email protected] > >> http://lists.pld-linux.org/mailman/listinfo/pld-cvs-commit > > > > -- > > Jan Rękorajski | PLD/Linux > > SysAdm | baggins<at>pld-linux.org | http://www.pld-linux.org/ > > _______________________________________________ > > pld-devel-en mailing list > > [email protected] > > http://lists.pld-linux.org/mailman/listinfo/pld-devel-en > > _______________________________________________ > pld-devel-en mailing list > [email protected] > http://lists.pld-linux.org/mailman/listinfo/pld-devel-en -- Jan Rękorajski | PLD/Linux SysAdm | baggins<at>pld-linux.org | http://www.pld-linux.org/ _______________________________________________ pld-devel-en mailing list [email protected] http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
