[Rpm-maint] [rpm-software-management/rpm] Proposal for default init.lua (#321)

2017-09-07 Thread Jason Tibbitts
-- Originally written by Jason Tibbitts <ti...@math.uh.edu> in 2016. -- Donated to the public domain. If you require a statement of license, please -- consider this work to be licensed as "CC0 Universal", any version you -- choose. -- My hope is that other distributions, or RPM it

Re: [Rpm-maint] [rpm-software-management/rpm] Proposal for default init.lua (#321)

2017-10-27 Thread Jason Tibbitts
My understanding is that the later definition simply overrides the former. But I would expect that conventions for namespacing would work to prevent accidental conflicts and if you can't redefine then you would just use functions in a namespace. I don't see why that would be mandatory, though

Re: [Rpm-maint] [rpm-software-management/rpm] Checks for Obsoletes (#385)

2018-01-25 Thread Jason Tibbitts
These are pretty minor but they do all seem to me to be reasonably good ideas. -- 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] Skip showing Group in output of `rpm -qi` (#534)

2018-08-27 Thread Jason Tibbitts
Various options: * Fedora could simply patch this downstream; it's just one popt file. * RPM could leave the Group field in the header empty instead of hardcoding "Unspecified". Then the popt file could just conditionally include the line. I've no idea what else would break, though. * The

Re: [Rpm-maint] [rpm-software-management/rpm] %setup -T -a 0 -a 1 doesn't extract %{S:0} (#462)

2018-08-29 Thread Jason Tibbitts
What about anything that calls `%autosetup`? You have to stop somewhere. Better would be to give sufficient expressiveness to the macro language to handle repeated arguments, and perhaps not need magic internal macros at all. Or maybe just add a Lua argument parsing library as standard and

Re: [Rpm-maint] [rpm-software-management/rpm] RFE: add some syntax to specify a macro should not fail when used with a flag not declared to rpm argument parsing (#547)

2018-09-27 Thread Jason Tibbitts
What we're trying to get around is having to do `%define foo(abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ)` so that the macro can just handle parsing `%**` itself, and then still not being able to handle anything that looks like a long option. That hack allows repeated arguments,

Re: [Rpm-maint] [rpm-software-management/rpm] RFE: make %{load: } work in %prep (#571)

2018-10-17 Thread Jason Tibbitts
But that only shows that @pmatilai's point was correct: If you need to do something dynamically in the build scriptlets, you need to do it with the shell. That bit of the spec does things via the shell. So, yes, it's clearly doable in some form: with the shell. I don't understand how what

Re: [Rpm-maint] [rpm-software-management/rpm] BUG Broken check for Name/Summary/Source0 existence (#572)

2018-10-17 Thread Jason Tibbitts
It would be far more useful if you could simply provide a few short specfiles which illustrate the problem you are having. I don't think you can reasonably expect the upstream RPM developers (or anyone else, really) to dig through that copr URL to try and figure out what in there might

Re: [Rpm-maint] [rpm-software-management/rpm] RFE Please add a verb to inhibit macro expansion (#569)

2018-10-17 Thread Jason Tibbitts
I though this might show it, but to me it looks like it works as expected; using either three or four percent signs seems correct. (That's kind of a fun result on its own, though.) Maybe your situation is more complicated or more nesting is required or something. That's why it's really

Re: [Rpm-maint] [rpm-software-management/rpm] Add support for sorting caret ('^') higher than base version (#88)

2018-11-15 Thread Jason Tibbitts
Hey, this was just pointed out to me, and it seems to perfectly provide a solution to a problem. As many might be aware, the Fedora packaging committee has periodically taken up the issue of using tilde Version:. And I've been putting in a load of effort trying to come up with a consistent

Re: [Rpm-maint] [rpm-software-management/rpm] RFE: don't prevealuate lua macros (#556)

2018-10-01 Thread Jason Tibbitts
Indeed, that's pretty much completely unpossible. Though if %luamacro takes no arguments and expands to shell code then of course that works to some degree. It may even work as you want to work. You just have to be very careful about where the line breaks are in the emitted shell code. --

Re: [Rpm-maint] [rpm-software-management/rpm] Proposal for default init.lua (#321)

2018-11-27 Thread Jason Tibbitts
Indeed, thanks. -- 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/321#issuecomment-442258681___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] RFE: %{_system_bindir} and such macro (#721)

2019-05-30 Thread Jason Tibbitts
This just reveals how redefining of %_bindir (or I guess more correctly they're redefining %_prefix) is not an ideal solution to the problem they're facing. I honestly have no problem with specifying a full path in a build dependency in the case that you can't for some reason specify a package

Re: [Rpm-maint] [rpm-software-management/rpm] macros: Add %version_notilde (#722)

2019-05-30 Thread Jason Tibbitts
This sort of seems the kind of thing that would go in to redhat-rpm-config (or another distro-specific package) first instead of straight upstream to RPM. But I guess I don't fully understand how new macros are expected to flow into RPM these days. -- You are receiving this because you are

Re: [Rpm-maint] [rpm-software-management/rpm] macros: Add %version_notilde (#722)

2019-05-30 Thread Jason Tibbitts
jasontibbitts commented on this pull request. > @@ -335,16 +335,16 @@ package or when debugging this package.\ # A colon separated list of desired locales to be installed; # "all" means install all locale specific files. -# Why give someone a reason to reject this just

Re: [Rpm-maint] [rpm-software-management/rpm] %buildarch does not reflect architecture of current %package declaration (#689)

2019-05-02 Thread Jason Tibbitts
Looking further, it appears to me that %buildarch isn't even defined until RPM sees `BuildArch: noarch` somewhere in the spec. At that point it gets defined to noarch and stays there. I don't know what utility that particular behavior has. -- You are receiving this because you are

[Rpm-maint] [rpm-software-management/rpm] %buildarch does not reflect architecture of current %package declaration (#689)

2019-05-02 Thread Jason Tibbitts
I'm trying to better understand how %buildarch works in the case of multiple %package declarations. For more context, see https://bugzilla.redhat.com/show_bug.cgi?id=1705656 Basically, it appears that as soon as BuildArch: noarch is seen anywhere, the value of %buildarch changes to "noarch"

Re: [Rpm-maint] [rpm-software-management/rpm] Unreadable directory causes difficult to debug failure (#776)

2019-07-11 Thread Jason Tibbitts
Thanks! -- 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/776#issuecomment-510657558___ Rpm-maint mailing list

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

[Rpm-maint] [rpm-software-management/rpm] pkgconfigdeps.sh and versions with spaces (#789)

2019-07-11 Thread Jason Tibbitts
I noticed a package (verilator) in Fedora that provides "2018-03-17". Turns out that it installs a pkgconfig file which contains `Version: 3.922 2018-03-17`, which causes `pkgconfigdeps.sh -P` to output `pkgconfig(verilator) = 3.922 2018-03-17`. Looking through various documents about

Re: [Rpm-maint] [rpm-software-management/rpm] pkgconfigdeps.sh and versions with spaces (#789)

2019-07-12 Thread Jason Tibbitts
Thanks, all. In the meantime the bizarre extra Provides: bit doesn't really hurt anything. One interesting side effect is that I'm not sure anything in Fedora would actually ever validate any of the installed pkgconf files. I wonder if this is a reasonable candidate for a brp script. (It

Re: [Rpm-maint] [rpm-software-management/rpm] Dependency generator for shell (#628)

2019-07-12 Thread Jason Tibbitts
I do think this could be useful if it would serve to reduce packager workload, though the trend seems to be moving away from scriptlets in general. -- 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] [RFE] Execute dependency generators on the .spec file which ships them (#782)

2019-07-12 Thread Jason Tibbitts
I don't believe it's that easy if you don't already have a `.attr` file. It doesn't look to me as if RPM will look at `%__foo_magic` or `%_foo_provides` unless it sees `%_fileattrsdir/foo.attr` first. Of course, you could override `%_fileattrsdir` instead, but then you would have to copy the

[Rpm-maint] [rpm-software-management/rpm] Unreadable directory causes difficult to debug failure (#776)

2019-06-28 Thread Jason Tibbitts
I recently helped track down a rather bizarre build failure, where rpmbuild exited 1 (causing mock to abort) but it wasn't really obvious why. Turns out that executable permissions had been removed from a directory. (Upstream had added a directory where previously there were only files, and

Re: [Rpm-maint] [rpm-software-management/rpm] Treat tilde versions as higher for purpose of dependency comparison (#715)

2019-08-07 Thread Jason Tibbitts
Since what you really want is a significantly different total ordering, you really need a different set of binary relations. I certainly don't want to be the one to propose `<~`, `<=~, `>~`, and `>=~`. Plus, for completeness, I guess you'd need a comparison operator `=~`. That would all

Re: [Rpm-maint] [rpm-software-management/rpm] Split verbose make flags to a separate macro (#799)

2019-07-31 Thread Jason Tibbitts
jasontibbitts commented on this pull request. > @@ -1073,7 +1073,8 @@ package or when debugging this package.\ #-- # The "make" analogue, hiding the _smp_mflags magic from specs -%make_build %{__make}

[Rpm-maint] [rpm-software-management/rpm] Split verbose make flags to a separate macro (#799)

2019-07-24 Thread Jason Tibbitts
If I didn't screw this up, it should help with #798. It just pulls the `V=1 VERBOSE=1` bit out to a separate macro to make it easier to override. You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/799 -- Commit Summary -- *

[Rpm-maint] [rpm-software-management/rpm] Addition of 'V=1' to %make_build broke a package (#798)

2019-07-24 Thread Jason Tibbitts
One of the packages I help maintain (apcupsd) broke oddly in rawhide with errors like this: ``` /bin/sh: 1man: command not found make[3]: 1g++: Command not found ``` Turns out this comes from the `V=1` which was added to `%make_build` in 8655493bdfd6b76271893b148033f2ff580d2d39. The software

Re: [Rpm-maint] [rpm-software-management/rpm] Make parametric macro arguments available as native Lua table (#1063)

2020-02-11 Thread Jason Tibbitts
Oh, this would be so great. -- 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/1063#issuecomment-585008636___ Rpm-maint mailing

Re: [Rpm-maint] [rpm-software-management/rpm] Implement build system templating in spec files (#1087)

2020-02-26 Thread Jason Tibbitts
Wow, this is quite interesting and somewhat similar to things I've brainstormed about for a number of years now I think that this has the possibility to simplify packaging for a significant portion of at least the Fedora packaging set. However, one problem we've struggled with is that when

Re: [Rpm-maint] [rpm-software-management/rpm] Disallow buildrequires: inside %package section (#1083)

2020-02-21 Thread Jason Tibbitts
I personally agree that it can be confusing, but I can't think of any technical reason why RPM would actively prevent it from working. I think this is more of a distribution issue; different distributions can choose to enforce different guidelines for stylistic issues such as these if they

Re: [Rpm-maint] [rpm-software-management/rpm] Axe defunct Lua rex extension (PR #1797)

2021-10-20 Thread Jason Tibbitts
I was just poking at this the other day after years of seeing that bit at the end of the RPM Lua documentation. Lua has reasonable pattern matching even if it's a bit weird, and it should suffice for most things. -- You are receiving this because you are subscribed to this thread. Reply to