[Debian-olpc-devel] review sugar-browse-activity

[Jonas Smedegaard]

Hi Ishan,

On Sun, Oct 17, 2010 at 07:59:18PM +0530, Ishan Bansal wrote:
>I have made a new repository for browse activity , because i thought of 
>getting my changes reviewed first before pushing them directly to the 
>browse repository.

>On Sun, Oct 17, 2010 at 2:18 AM, Jonas Smedegaard <dr at jones.dk> wrote:
>> On Sat, Oct 16, 2010 at 10:49:23PM +0530, Ishan Bansal wrote:
>>
>>> I have updated the browse activity and made a new git repository as 
>>> sugar-browser-activity.
>>>
>>
>> Why?

[please consider top-writing[1] - it makes quoting difficult to read]

Ah, I see.

Creating short-lived gits only for the purpose of review is not a good 
approach in my opinion, though.

I suggest the following instead:

  1) if you worry that others here might disagree with your changes,
     then first discuss here on the list what you intend to change,
     before you do work that might be a waste of your time

  2) check out the main git, and prepare your changes locally

  3) test your changes locally, that they do what you intended, and
     that the packaging still works properly.
     Preferrably, especially when working on packagings you are not
     normally working on (as is the case with the Browse activity),
     compare your resulting packaging build against latest official
     release of the packaging (e.g. using dget) and compare them
     (e.g. using debdiff).

  4) if you run into trouble with your work locally, then don't
     hesitate to ask for help here on the list.  No questions are
     stupid! Even foolish seeming question, others can learn from!
     If you need to share with us some details of your local changes
     then try attach a patch as attachment to your email to list
     (e.g. using git diff or git log -p).

  5) when you feel confident that your changes are ok locally, then
     push to the main shared git.
     If we disagree with your changes, they are easy to revert later.
     Or more accurately: simple things are easy to revert.
     Please think twice and strongly consider to ask on the list first
     before you upgrade to a newer upstream release or other changes
     involving adding tags, as they cannot be easily reverted.


I can't force you, but I would really really appreciate if more of our 
discussions happened here on the list!


Kind regards,

  - Jonas


[1] Info on top-posting: 
https://secure.wikimedia.org/wikipedia/en/wiki/TOFU

-- 
  * Jonas Smedegaard - idealist & Internet-arkitekt
  * Tlf.: +45 40843136  Website: http://dr.jones.dk/

  [x] quote me freely  [ ] ask before reusing  [ ] keep private
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/debian-olpc-devel/attachments/20101017/21fba5d9/attachment.pgp>