[Rpm-maint] [PATCH 1/2] Extending rpm plugin interface, part 1
pmatilai at laiskiainen.org
Thu Nov 8 10:39:45 UTC 2012
On 11/08/2012 08:35 AM, Reshetova, Elena wrote:
> Sorry for the late reply: I was on holidays.
No worries. Lucky you :)
> Sure, go ahead with rpmlog and
> indentation changes, if it doesn't bother you to do this!
Okay then, done and pushed. Now that I looked closer, I spotted (and
fixed) a couple of more "issues": a tiny memleak from early
rpmtsSetupTransactionPlugins() return and some further cosmetics (two
soft-tabs instead of one hard-tab, trailing whitespace etc), but nothing
Oh and one other thing I noticed just now that'll need further thought:
currently the script setup hook only runs for external scripts, but not
the embedded Lua-scripts. Which are getting more and more common...
They'll obviously need to be handled quite differently as they run
within the rpm process itself, ie fork() + exec() does not occur.
> I will then start concentrating on rest of the stuff: need to do some more
> thinking on it to begin with.
Cool. And thanks for all the work so far :)
- Panu -
More information about the Rpm-maint