[U-Boot] [PATCH v2] intern: Add copy scripts and target specific helpers

[Heiko Schocher]

Hello Hannes,

Am 04.02.2015 08:10, schrieb Hannes Petermaier:
> Heiko,
>
> what a great tool :-)

Yep, definitely... Thanks to Simon!

> This makes the whole thing much easier and will produce (hopefully) less
> noise from me :-)

;-)

> But i 've a further questions:
>
> 1st)
> patman prints out some errors, looks like for me that something is wrong
> with the first line of my commit message - but no idea what.
>
> "
> Traceback (most recent call last):
>    File "tools/patman/patman", line 149, in <module>
>      options.add_maintainers)
>    File "/home/petermaierh/work/u-boot/tools/patman/series.py", line 227,
> in MakeCcFile
>      raise_on_error=raise_on_error)
>    File "/home/petermaierh/work/u-boot/tools/patman/gitutil.py", line 321,
> in BuildEmailList
>      raw += LookupEmail(item, alias, raise_on_error=raise_on_error)
>    File "/home/petermaierh/work/u-boot/tools/patman/gitutil.py", line 490,
> in LookupEmail
>      raise ValueError, msg
> ValueError: Alias 'drivers/video/am335x-fb' not found
> "
> maybe you've or somebody else has an idea how to fix this ?

I am not really a python coder ... and I do not see such an error,
if I create a commit with changes in "drivers/video/am335x-fb.c"
I got not this error ... my first thought it was the
"drivers/video/am335x-fb:" in your subject line... but I used the same
subject for a test commit ... please try another subject line:
"drivers, video, am335x: ..." ?

How does your patman config file looks like?

> 2nd)
> How to achieve "not breaking threading" ? How the right "in-reply-to" is
> added to git-send-email ? from where comes this information.
> I've seen at my last V2 that i don't have reached the goal.

Hmm... I am not sure, maybe this is a ToDo? Simon? Could you
help here?

Thanks!

bye,
Heiko
>
> many thanks and best regards,
> Hannes
>
> Heiko Schocher <hs at denx.de> schrieb am 03.02.2015 13:47:02:
>
>> From: Heiko Schocher <hs at denx.de>
>> To: Hannes Petermaier <Hannes.Petermaier at br-automation.com>
>> Cc: Hannes Petermaier <oe5hpm at oevsv.at>, u-boot at lists.denx.de, U-Boot
> <u-boot-
>> bounces at lists.denx.de>
>> Date: 03.02.2015 13:47
>> Subject: Re: [U-Boot] [PATCH v2] intern: Add copy scripts and target
> specific helpers
>>
>> Hello Hannes,
>>
>> Am 03.02.2015 13:37, schrieb Hannes Petermaier:
>>> Hi all,
>>>
>>> please ignore this "internal" patch.
>>> This is was an accident.
>>
>> Maybe it is worth to look into patman, see:
>> u-boot:/tools/patman/README ?
>>
>> bye,
>> Heiko
>>>
>>> best regards,
>>> Hannes
>>>
>>> _______________________________________________
>>> U-Boot mailing list
>>> U-Boot at lists.denx.de
>>> http://lists.denx.de/mailman/listinfo/u-boot
>>>
>>
>> --
>> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
>> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
>
>
>

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany