Patchwork [bitbake-devel,2/2] bitbake-user-manual-intro.xml: update for dump-signatures

login
register
mail settings
Submitter Robert Yang
Date July 4, 2014, 5:39 a.m.
Message ID <29052f940ce9526083a09d3ce4e665a5aa9e0db1.1404452300.git.liezhi.yang@windriver.com>
Download mbox | patch
Permalink /patch/74989/
State New
Headers show

Comments

Robert Yang - July 4, 2014, 5:39 a.m.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 .../bitbake-user-manual-intro.xml                  |   10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)
Rifenbark, Scott M - July 7, 2014, 6:13 a.m.
Patch applied, 

Thanks, 
Scott

>-----Original Message-----
>From: bitbake-devel-bounces@lists.openembedded.org [mailto:bitbake-
>devel-bounces@lists.openembedded.org] On Behalf Of Robert Yang
>Sent: Thursday, July 03, 2014 10:39 PM
>To: bitbake-devel@lists.openembedded.org
>Subject: [bitbake-devel] [PATCH 2/2] bitbake-user-manual-intro.xml: update
>for dump-signatures
>
>Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>---
> .../bitbake-user-manual-intro.xml                  |   10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
>diff --git a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>index ae267b4..e4025a8 100644
>--- a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>+++ b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>@@ -471,11 +471,13 @@
>        -D, --debug           Increase the debug level. You can specify this more
>                              than once.
>        -n, --dry-run         Don't execute, just go through the motions.
>-       -S DUMP_SIGNATURES, --dump-signatures=DUMP_SIGNATURES
>+       -S SIGNATURE_HANDLER, --dump-signatures=SIGNATURE_HANDLER
>                              Dump out the signature construction information, with
>-                             no task execution. Parameters are passed to the
>-                             signature handling code, use 'none' if no specific
>-                             handler is required.
>+                             no task execution. The SIGNATURE_HANDLER can be none
>+                             or printdiff, while none means only dump the
>+                             signature, printdiff means compare the dumped
>+                             signature with the cached one.
>+
>        -p, --parse-only      Quit after parsing the BB recipes.
>        -s, --show-versions   Show current and preferred versions of all recipes.
>        -e, --environment     Show the global or per-package environment
>complete
>--
>1.7.9.5
>
>--
>_______________________________________________
>bitbake-devel mailing list
>bitbake-devel@lists.openembedded.org
>http://lists.openembedded.org/mailman/listinfo/bitbake-devel
Robert Yang - July 7, 2014, 6:16 a.m.
On 07/07/2014 02:13 PM, Rifenbark, Scott M wrote:
> Patch applied,
>

Sorry, this has not been approved, we need wait for RP's comments and update
it, and we should merge the part for bitbake first.

// Robert

> Thanks,
> Scott
>
>> -----Original Message-----
>> From: bitbake-devel-bounces@lists.openembedded.org [mailto:bitbake-
>> devel-bounces@lists.openembedded.org] On Behalf Of Robert Yang
>> Sent: Thursday, July 03, 2014 10:39 PM
>> To: bitbake-devel@lists.openembedded.org
>> Subject: [bitbake-devel] [PATCH 2/2] bitbake-user-manual-intro.xml: update
>> for dump-signatures
>>
>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>> ---
>> .../bitbake-user-manual-intro.xml                  |   10 ++++++----
>> 1 file changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>> b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>> index ae267b4..e4025a8 100644
>> --- a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>> +++ b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>> @@ -471,11 +471,13 @@
>>         -D, --debug           Increase the debug level. You can specify this more
>>                               than once.
>>         -n, --dry-run         Don't execute, just go through the motions.
>> -       -S DUMP_SIGNATURES, --dump-signatures=DUMP_SIGNATURES
>> +       -S SIGNATURE_HANDLER, --dump-signatures=SIGNATURE_HANDLER
>>                               Dump out the signature construction information, with
>> -                             no task execution. Parameters are passed to the
>> -                             signature handling code, use 'none' if no specific
>> -                             handler is required.
>> +                             no task execution. The SIGNATURE_HANDLER can be none
>> +                             or printdiff, while none means only dump the
>> +                             signature, printdiff means compare the dumped
>> +                             signature with the cached one.
>> +
>>         -p, --parse-only      Quit after parsing the BB recipes.
>>         -s, --show-versions   Show current and preferred versions of all recipes.
>>         -e, --environment     Show the global or per-package environment
>> complete
>> --
>> 1.7.9.5
>>
>> --
>> _______________________________________________
>> bitbake-devel mailing list
>> bitbake-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/bitbake-devel
>
>
Rifenbark, Scott M - July 7, 2014, 6:18 a.m.
Ok - if there is a problem or changes then let me know.  I have the doc changes pushed to poky-contrib/srifenbark/bb-manual only.  Richard would have to merge them from there into the upstream bitbake repository.

Scott

>-----Original Message-----
>From: Robert Yang [mailto:liezhi.yang@windriver.com]
>Sent: Sunday, July 06, 2014 11:16 PM
>To: Rifenbark, Scott M; bitbake-devel@lists.openembedded.org
>Subject: Re: [bitbake-devel] [PATCH 2/2] bitbake-user-manual-intro.xml:
>update for dump-signatures
>
>
>
>On 07/07/2014 02:13 PM, Rifenbark, Scott M wrote:
>> Patch applied,
>>
>
>Sorry, this has not been approved, we need wait for RP's comments and
>update it, and we should merge the part for bitbake first.
>
>// Robert
>
>> Thanks,
>> Scott
>>
>>> -----Original Message-----
>>> From: bitbake-devel-bounces@lists.openembedded.org [mailto:bitbake-
>>> devel-bounces@lists.openembedded.org] On Behalf Of Robert Yang
>>> Sent: Thursday, July 03, 2014 10:39 PM
>>> To: bitbake-devel@lists.openembedded.org
>>> Subject: [bitbake-devel] [PATCH 2/2] bitbake-user-manual-intro.xml:
>>> update for dump-signatures
>>>
>>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>>> ---
>>> .../bitbake-user-manual-intro.xml                  |   10 ++++++----
>>> 1 file changed, 6 insertions(+), 4 deletions(-)
>>>
>>> diff --git
>>> a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>>> b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>>> index ae267b4..e4025a8 100644
>>> --- a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>>> +++ b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
>>> @@ -471,11 +471,13 @@
>>>         -D, --debug           Increase the debug level. You can specify this more
>>>                               than once.
>>>         -n, --dry-run         Don't execute, just go through the motions.
>>> -       -S DUMP_SIGNATURES, --dump-signatures=DUMP_SIGNATURES
>>> +       -S SIGNATURE_HANDLER, --dump-signatures=SIGNATURE_HANDLER
>>>                               Dump out the signature construction information, with
>>> -                             no task execution. Parameters are passed to the
>>> -                             signature handling code, use 'none' if no specific
>>> -                             handler is required.
>>> +                             no task execution. The SIGNATURE_HANDLER can be none
>>> +                             or printdiff, while none means only dump the
>>> +                             signature, printdiff means compare the dumped
>>> +                             signature with the cached one.
>>> +
>>>         -p, --parse-only      Quit after parsing the BB recipes.
>>>         -s, --show-versions   Show current and preferred versions of all
>recipes.
>>>         -e, --environment     Show the global or per-package environment
>>> complete
>>> --
>>> 1.7.9.5
>>>
>>> --
>>> _______________________________________________
>>> bitbake-devel mailing list
>>> bitbake-devel@lists.openembedded.org
>>> http://lists.openembedded.org/mailman/listinfo/bitbake-devel
>>
>>
Richard Purdie - July 21, 2014, 6:18 p.m.
On Mon, 2014-07-07 at 06:18 +0000, Rifenbark, Scott M wrote:
> Ok - if there is a problem or changes then let me know.  I have the
> doc changes pushed to poky-contrib/srifenbark/bb-manual only.  Richard
> would have to merge them from there into the upstream bitbake
> repository.

I modified Roberts patch a bit. Could you update the manual based on
that patch? I didn't take the change for the manual you have queued for
this, I'd suggest we create a new one. I did merge everything else in
your bitbake manual branch.

Cheers,

Richard
Rifenbark, Scott M - July 22, 2014, 6:50 a.m.
Richard, 

I pulled 'poky/master' to be sure I had latest code and reproduced the 'bitbake -h' output to see what is there.  From that, I updated the example output in the bitbake manual and pushed to contrib at srifenbark/bb-manual.  Along with that updated dump signatures stuff I also noted that the option '--token=XMLRPCTOKEN' appeared to not have been in the help output in the manual.  So, because the stuff in the manual is an exact copy of the actual help output, I have refreshed the example with the actual output, which includes --token as well.  

The changes are pushed to contrib on srifenbark/bb-manual.  Notice also that Robert (30 minutes prior to my push) also has a commit that fixed the dump signature stuff I suspect based on your email to this group.  You should probably merge my commit as it contains the --token option and the fixed dump signature stuff. 

Scott

>-----Original Message-----
>From: Richard Purdie [mailto:richard.purdie@linuxfoundation.org]
>Sent: Monday, July 21, 2014 11:19 AM
>To: Rifenbark, Scott M
>Cc: Yang, Liezhi (Wind River); bitbake-devel@lists.openembedded.org
>Subject: Re: [bitbake-devel] [PATCH 2/2] bitbake-user-manual-intro.xml:
>update for dump-signatures
>
>On Mon, 2014-07-07 at 06:18 +0000, Rifenbark, Scott M wrote:
>> Ok - if there is a problem or changes then let me know.  I have the
>> doc changes pushed to poky-contrib/srifenbark/bb-manual only.  Richard
>> would have to merge them from there into the upstream bitbake
>> repository.
>
>I modified Roberts patch a bit. Could you update the manual based on that
>patch? I didn't take the change for the manual you have queued for this, I'd
>suggest we create a new one. I did merge everything else in your bitbake
>manual branch.
>
>Cheers,
>
>Richard
Robert Yang - July 22, 2014, 8:12 a.m.
On 07/22/2014 02:18 AM, Richard Purdie wrote:
> On Mon, 2014-07-07 at 06:18 +0000, Rifenbark, Scott M wrote:
>> Ok - if there is a problem or changes then let me know.  I have the
>> doc changes pushed to poky-contrib/srifenbark/bb-manual only.  Richard
>> would have to merge them from there into the upstream bitbake
>> repository.
>
> I modified Roberts patch a bit. Could you update the manual based on
> that patch? I didn't take the change for the manual you have queued for
> this, I'd suggest we create a new one. I did merge everything else in
> your bitbake manual branch.

Sorry, but I think that we need remove this:

type="choice", choices=("none", "printdiff")

after your explanations, otherwise we can't input other handlers.

I will update it later.

// Robert

>
> Cheers,
>
> Richard
>
>
>
Richard Purdie - July 22, 2014, 8:35 a.m.
On Tue, 2014-07-22 at 16:12 +0800, Robert Yang wrote:
> 
> On 07/22/2014 02:18 AM, Richard Purdie wrote:
> > On Mon, 2014-07-07 at 06:18 +0000, Rifenbark, Scott M wrote:
> >> Ok - if there is a problem or changes then let me know.  I have the
> >> doc changes pushed to poky-contrib/srifenbark/bb-manual only.  Richard
> >> would have to merge them from there into the upstream bitbake
> >> repository.
> >
> > I modified Roberts patch a bit. Could you update the manual based on
> > that patch? I didn't take the change for the manual you have queued for
> > this, I'd suggest we create a new one. I did merge everything else in
> > your bitbake manual branch.
> 
> Sorry, but I think that we need remove this:
> 
> type="choice", choices=("none", "printdiff")
> 
> after your explanations, otherwise we can't input other handlers.
> 
> I will update it later.

Right, yes, other values are/will be allowed. Thanks!

Richard

Patch

diff --git a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
index ae267b4..e4025a8 100644
--- a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
+++ b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-intro.xml
@@ -471,11 +471,13 @@ 
        -D, --debug           Increase the debug level. You can specify this more
                              than once.
        -n, --dry-run         Don't execute, just go through the motions.
-       -S DUMP_SIGNATURES, --dump-signatures=DUMP_SIGNATURES
+       -S SIGNATURE_HANDLER, --dump-signatures=SIGNATURE_HANDLER
                              Dump out the signature construction information, with
-                             no task execution. Parameters are passed to the
-                             signature handling code, use 'none' if no specific
-                             handler is required.
+                             no task execution. The SIGNATURE_HANDLER can be none
+                             or printdiff, while none means only dump the
+                             signature, printdiff means compare the dumped
+                             signature with the cached one.
+
        -p, --parse-only      Quit after parsing the BB recipes.
        -s, --show-versions   Show current and preferred versions of all recipes.
        -e, --environment     Show the global or per-package environment complete