[Rpm-maint] [rpm-software-management/rpm] RFE: Permit %load/%expand recursive expansion while reading a macrofile (#410)

2018-03-06 Thread Jeff Johnson
A very minor patch to permit immediate expansion of %load/%expand while parsing 
key/value configuration in static macro file configuration would usefully 
increase the flexibility of rpmbuild configuration.

All I am suggesting is to handle %{load:...} and %{expand:...} as special cases 
while reading configuration.

The boring problem of infinite %load recursion if, say, a macrofile tries to 
load itself in a loop can be prevented by testing the recursion depth and 
error'ing out with a message if %load is looping

-- 
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/410___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: Configurable macro file search path(s) (#363)

2018-03-06 Thread Jeff Johnson
RPM uses macros instead of envvars to increase reproducibility of builds.

How rpm is configured, and how rpmbuild is invoked, are matters that you need 
to take up with build systems. That would also be true if envvar's were used 
instead of macros.

Note that there is (or at least was) a 3rd way to change macros to load while 
building, using a macro files directive in rpmrc configuration. I dunno whether 
that mechanism is still supported or recommended.

In summary, you now have 3 mechanisms, 2 of which permit patterns, to customize 
rpmbuild to taste.

You need to address further questions with build system maintainers, not rpm.

-- 
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/363#issuecomment-370917574___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] Reserve ALT RPM tags range

2018-03-06 Thread Vladimir D. Seleznev
On Wed, Feb 28, 2018 at 05:54:27AM -0500, Jeff Johnson wrote:
> > You might try the convention of assigning tags in the 1Gb space from 
> > 0x4000 to 0x4fff as you wish.
> > 
> > (aside)
> > RPM5 has something called "arbitrary tags" in the 0x4000 -> 0x4fff 
> > range.
> > 
> > Short answer:
> > Choose any tagno in that range and do what you wish.
> > 
> > Longer answer:
> > 
> > Choose a tag name string.
> > 
> > The  tagno is computed from a (configurable) tag name string as follows:
> > 
> > 1) the tag name plain text is canonicalized (leading alphabetic, 
> > alphanumeric characters, 1st letter uppercase, rest lowercase). E.g. 
> > "Mynewtag42"
> > 
> > 2) the  4 binary bytes of the SHA1 of the canonical string are copied into 
> > a uint32 (which needs to be swabbed on big endian platforms).
> > 
> > 3) the tagno is assigned by then masking on the 0x4000 arbitrary tag 
> > identifier onto the least significant 30 bits of the uint32.

Thank you for the suggestion, I think we shall follow this!

-- 
   With best regards,
   Vladimir D. Seleznev
___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: Configurable macro file search path(s) (#363)

2018-03-06 Thread Jeff Johnson
Alternatively, choose known file names, and use a %load for each file in a spec 
file.

-- 
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/363#issuecomment-370833625___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: Configurable macro file search path(s) (#363)

2018-03-06 Thread Jeff Johnson
Then invoke rpmbuild with --macros, appending your patterns at the end of the 
colon separated list.

-- 
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/363#issuecomment-370831841___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: Configurable macro file search path(s) (#363)

2018-03-06 Thread Vít Ondruch
I am looking for a way to tell RPM that there might be some paths which might 
contain arbitrary macro files. If such files are located, they should be loaded 
by RPM.

IOW given there is metapackage like this \[[1]\] and I have installed 
rh-ruby22-runtime package and I call ```scl enable rh-ruby22 bash```, in this 
environment, I'd like RPM to be able to pickup macro files which might be 
located in ```/opt/rh/rh-ruby22/root/urs/lib/rpm/macros.d/``` directory.

[1]: 
https://copr-be.cloud.fedoraproject.org/results/rhscl/rh-ruby22-el7/epel-7-x86_64/rh-ruby22-2.0-6.el7/

-- 
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/363#issuecomment-370796366___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: Configurable macro file search path(s) (#363)

2018-03-06 Thread Jeff Johnson
What exactly are you looking for?

You have yet to post any example here that does not include a pattern when the 
documentation for %load explicitly says "file".

Have you tried strace to verify that rpm is trying to load a file?

-- 
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/363#issuecomment-370786636___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: Configurable macro file search path(s) (#363)

2018-03-06 Thread Vít Ondruch
I am not surprised at all that the wildcard is not supported. I am just 
surprised I am suggested to use ```%load``` where it is not usable in its 
current form.

-- 
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/363#issuecomment-370702007___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint