Copying ubuntu-sugar list serv for archiving the reviews on sugar-moon-activity.<div><br></div><div>Regards,</div><div><br></div><div>Manu<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Ankur Khurana</b> <span dir="ltr"><<a href="mailto:ankur@seeta.in">ankur@seeta.in</a>></span><br>
Date: Sat, Aug 7, 2010 at 9:03 AM<br>Subject: Re: ITP: sugar-moon-activity -- Moon is a simple Lunar phase activity for Sugar, including upcoming Lunar eclipse information and effects.<br>To: Ishan Bansal <<a href="mailto:ishan@seeta.in">ishan@seeta.in</a>><br>
Cc: Neeraj Gupta <<a href="mailto:neeraj@seeta.in">neeraj@seeta.in</a>>, Manusheel Gupta <<a href="mailto:manu@seeta.in">manu@seeta.in</a>>,<br><br><br>Review of activity based on commit:on initial commit:<br>
1.There is no pristine branch.When pushing use --all after git push and before the address to be pushed.I have updated the google doc, please take a look at it as well.<br>
<br>2.dont use © in copyright file. and give ',' after the year and before the copyright holder.check copyright_hints to check any other copyright holder.<br><br>3.order of files in copyright is:<br><br>
1) default - i.e. "*" or the Files: lines left out completely<br>
2) subdir match using wildcard<br>
3) file match<br>
4) explicit path match<br>
5) ./debian/* match<br>
6) debian subdir match - e.g. ./debian/patches/*<br>please try to follow this order.<br><br>4.Always, Always run clean rule before you add anything to git or upload it.It removes lot of uneccesaary files.<br><br>5.In Control file, always wrap your description at 72 characters per liune.if use next line to complete the description.You might like to change sugar moon activity to Sugar Moon Activity in description in control file.<br>
<br>6 In README.sources file, you might like to remove from line 19 to 74 as it does not concerns our packages as of now.<br><br>7.Well your format of giving url in watch file is wrong.<br>It should be like this : <a href="http://download.sugarlabs.org/sources/honey/Moon/Moon-(.*)%5C.tar%5C.bz2" target="_blank">http://download.sugarlabs.org/sources/honey/Moon/Moon-(.*)\.tar\.bz2</a><br>
<br>I think, make these changes first, will have a other look aftere sometime.<div class="im"><br><br><div class="gmail_quote">On Sat, Aug 7, 2010 at 2:10 AM, Ishan Bansal <span dir="ltr"><<a href="mailto:ishan@seeta.in" target="_blank">ishan@seeta.in</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left:1px solid rgb(204, 204, 204);margin:0pt 0pt 0pt 0.8ex;padding-left:1ex">plz review the package-: sugar-moon-activity<br><br></blockquote></div></div>Regards,<br><font color="#888888">Ankur<br>
</font></div><br></div>