From: Otavio Salvador <otavio@ossystems.com.br>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH] New package: siproxd
Date: Mon, 16 Feb 2009 09:21:30 -0300 [thread overview]
Message-ID: <873aeeh8s5.fsf@neumann.lab.ossystems.com.br> (raw)
In-Reply-To: <200902161228.26942.khimov@altell.ru> (Roman I. Khimov's message of "Mon, 16 Feb 2009 12:28:26 +0300")
Roman I Khimov <khimov@altell.ru> writes:
> Otavio Salvador:
>> Basically we could suggest following workflow for regular users:
>>
>> - when going to make any change at OE, start a personal branch
>>
>> $: git checkout -b local master
>
> Here is when the problem starts. Currently we have overlay setup, so I'm
> extracting everything I'm pushing now from overlay (which itself is managed
> in git, but that's another story since at least one thing is not managed at
> all in overlay - checksums).
>
>> $: git format-patch master..
>>
>> - mail them to ml
>
> And this is another thing I'm not sure about, there might be a lot
> of 'intermediate' commits leading to nice-upstream-acceptable package,
> pushing them all in series is kinda wrong, IMHO.
>
> There is of course some git kung fu that might help doing one patch for one
> thing regardless of the number of intermediate results. But then again
> there is reviewing process that leads to more changes and more git kung fu
> to make single patch for single package again.
>
> Does it make any sense? Currently I'm trying to push new things in 'one
> package - one patch' model, extracting package patch from overlay as `git
> diff some_commit..` plus adding checksums to local OE tree and doing `git
> diff` there (not commiting it at all). Then after new package is in, I'm
> able to pull from main OE and remove things from overlay.
You need to created another temp branch and cherry-pick the changes to
it; after grabbing them you can use:
$: git format-patch master..temp
--
Otavio Salvador O.S. Systems
E-mail: otavio@ossystems.com.br http://www.ossystems.com.br
Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
next prev parent reply other threads:[~2009-02-16 12:23 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-13 21:35 [PATCH] New package: siproxd Roman I Khimov
2009-02-14 6:30 ` Marcin Juszkiewicz
2009-02-14 8:34 ` Koen Kooi
2009-02-14 14:15 ` Roman I Khimov
2009-02-15 18:23 ` Otavio Salvador
2009-02-15 21:13 ` Tom Rini
2009-02-16 1:09 ` Otavio Salvador
2009-02-16 9:28 ` Roman I Khimov
2009-02-16 10:22 ` Marcin Juszkiewicz
2009-02-16 12:21 ` Otavio Salvador [this message]
2009-02-17 12:25 ` Roman I Khimov
2009-02-27 4:34 ` Khem Raj
2009-03-25 20:21 ` Roman I Khimov
2009-03-23 8:26 ` Otavio Salvador
2009-03-23 18:37 ` Roman I Khimov
2009-03-23 20:25 ` Otavio Salvador
2009-03-24 9:47 ` Koen Kooi
2009-03-26 14:56 ` Roman I Khimov
2009-04-21 20:17 ` Roman I Khimov
2009-04-21 20:29 ` Koen Kooi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=873aeeh8s5.fsf@neumann.lab.ossystems.com.br \
--to=otavio@ossystems.com.br \
--cc=openembedded-devel@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.