Re: [Rpm-maint] [rpm-software-management/rpm] Handle Python extras trough reverse requirements encoded in provides (#1061)

2020-02-12 Thread Gordon Messmer
> Note that the subpackages approach has a problem if an extra goes away and it > was installed on user machine. Maybe not a *huge* concern... setuptools' documentation tells developers not to do that.

Re: [Rpm-maint] [rpm-software-management/rpm] Make "%patchlist -f patches" work. v2 (#1043)

2020-02-12 Thread Florian Festi
Ok, this should be it. I am going to squash all but the first patches into one after review. -- 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] Make "%patchlist -f patches" work. v2 (#1043)

2020-02-12 Thread Florian Festi
ffesti commented on this pull request. > + rpmlog(RPMLOG_ERR, _("line %d: Bad option %s: %s\n"), + spec->lineNum, + poptBadOption(optCon, POPT_BADOPTION_NOALIAS), + spec->line); + goto exit; +} + +lst = argvNew(); + +for (arg = 1; arg

Re: [Rpm-maint] [rpm-software-management/rpm] Make "%patchlist -f patches" work. v2 (#1043)

2020-02-12 Thread Florian Festi
@ffesti pushed 1 commit. 088636d631850bd91bdd267d8f96bc568596ef04 Use popt to properly parse the arguments of %patchlist and %sourcelist -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Implement %{body:...} built-in for retrieving the literal macro body (#1064)

2020-02-12 Thread Panu Matilainen
I can live with macrobody, a little bit of name-spacing never hurt anybody. In the meanwhile, I'm starting to think it does need to expand its argument like everything else, this is too limiting. Ie it'd still take a macro name as an argument, but that the name can come from a macro instead of

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

2020-02-12 Thread Panu Matilainen
Okay, at least someone thinks this would be useful, so I guess it makes it worth pursuing :slightly_smiling_face: The implementation here is a icky and slow as it has to go backwards and out of its way to reconstruct what grabArgs() had readily available, probably grabArgs() just needs to

Re: [Rpm-maint] [rpm-software-management/rpm] Add a --salvagedb option to the rpmdb tool (#1042)

2020-02-12 Thread Panu Matilainen
Right, I'm by no means opposed to this, and integrating with rpmdb is certainly the way to go, rather than strange external tools. Was just curious about the concrete difference. Can you update the commit message to elaborate a bit on how it works and possible side-effects, at least that part

Re: [Rpm-maint] [rpm-software-management/rpm] Add a --salvagedb option to the rpmdb tool (#1042)

2020-02-12 Thread Michael Schroeder
@mlschroe pushed 1 commit. dd4bb9f38fdf1d147a09ca0cdbf38ed75e409b17 Add a --salvagedb option to the rpmdb tool -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Implement %{body:...} built-in for retrieving the literal macro body (#1064)

2020-02-12 Thread Michael Schroeder
I'd prefer if we named this %macrobody, so that there's less chance that it clashes with somebody else's use of %body. -- 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] Reset global macro state after each spec query/parse (#1067)

2020-02-12 Thread Florian Festi
Merged #1067 into master. -- 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/1067#event-3029980388___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] Add a --salvagedb option to the rpmdb tool (#1042)

2020-02-12 Thread Michael Schroeder
No, there's a (very) small chance that it picks up entries that have been deleted. It's like the .recover command of the sqlite cli: Like the ".dump" command, ".recover" attempts to convert the entire contents of a database file to text. The difference is that instead of reading data using the

Re: [Rpm-maint] [rpm-software-management/rpm] Reset global macro state after each spec query/parse (#1067)

2020-02-12 Thread Panu Matilainen
Those incidents are probably related: http://lists.rpm.org/pipermail/rpm-list/2020-February/002011.html -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: