[Rpm-maint] Plugin ponderings

[Reshetova, Elena]

>Much better late than never, the indentation is now exactly as it should
be, good! There's just one extra leftover indent level for the execv() call.
Oh, missed that one since it wasn't in diff: fixed now!

>I'm afraid we'll need one more round: you perhaps took my "that's what
RPMSCRIPTLET_EXEC bit is for" (wrt embedded scripts) a bit too literally :)
>What I meant is that the RPMSCRIPTLET_EXEC bit defines whether something is
embedded or external - external scripts always have it, embedded ones never
have it as there is nothing to exec. The patch gets it a bit backwards in
rpmScriptRun():

 Oh, then I indeed misunderstood you: I was thinking that external scripts
will have fork + exec and lua ones just exec, since they are "executed"
after all (just not via exec). Now fixed, too. 

>While not strictly necessary, I think rpmLuaScript() could be changed to
take the additional 'plugins' argument while we're at it, if only to keep
symmetry between the two functions. It'll be needed later on anyway, unlike
the selinux argument which gets >passed to runLuaScript() just for the
symmetry's sake.

Sure, added!

I am attaching the new version now *without* saying that this is "the last
one" :) 

Best Regards,
Elena.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Improving-scriptlet-related-rpm-plugin-hooks.patch
Type: application/octet-stream
Size: 9421 bytes
Desc: not available
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20121129/a372beb3/attachment-0001.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 7220 bytes
Desc: not available
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20121129/a372beb3/attachment-0001.p7s>