[pacman-dev] [PATCH] makepkg: move option parsing code to separte file
Allan McRae
allan at archlinux.org
Tue Jun 14 05:24:38 EDT 2011
On 14/06/11 09:48, Dan McGee wrote:
> On Sat, Jun 11, 2011 at 11:32 PM, Allan McRae<allan at archlinux.org> wrote:
>> On 11/06/11 07:18, Dan McGee wrote:
>>> We don't actually have any python scripts anymore I just realized...
>>> I'll kill the cruft in another patch.
>>
>> I have a patch for that but was deciding what to do with it... Maybe this
>> should be kept in case we do get a python (or other language) script as a
>> template for how to deal with it.
> If you got the patch, go ahead and throw it in your working branch
> whenever- I'd take the axe to it; if we need to bring it back git
> history is always there for us.
A bunch of stuff should be pushed there in the next few days. There are
some advantages of being stuck in hospital! I'll add that patch to it.
>>> maybe a
>>> local scriptname="@@SCRIPTNAME@@"
>>> once here to avoid scattering it all over the place?
>>>
>>
>> OK. Not that it really matters give it is all one sed to replace it.
> Yeah, I don't know why my gut says to do it this way- I guess just the
> less magic the better, and for most of these kind of replacements we
> try to do it this way. It also prevents silly mistakes like including
> @@dasfsdf@@ in a gettext-ized string.
>
My gut is saying to go the other way. :P Anyway, I'll really have no
objection either way so I will change it.
Allan
More information about the pacman-dev
mailing list