On Tue, Feb 15, 2011 at 12:25 AM, Wizard <[email protected]> wrote: > Correction: Forget to do a -u. :) > > *Name: Simplicity > *addon: skin.simplicity > *version: 1.0.3 > *url: git://github.com/wizardxbmcmods/simplicity-xbmc > *revision - ac1e4d1e > *branch - master > *xbmc version - dharma > > -------------------------------------------------- > From: "Cory Fields" <[email protected]> > Sent: Monday, February 14, 2011 10:52 PM > To: "Wizard" <[email protected]> > Cc: <[email protected]> > Subject: Re: [Xbmc-addons] [Git Pull] [skin.simplicity] Initial pull request > >> On Mon, Feb 14, 2011 at 11:08 PM, Wizard <[email protected]> wrote: >>> >>> *Name: Simplicity >>> *addon: skin.simplicity >>> *version: 1.0.2 >>> *url: git://github.com/wizardxbmcmods/simplicity-xbmc >>> *revision - 32937652 >>> *branch - master >>> *xbmc version - dharma >>> >>> -------------------------------------------------------------------------------------- >>> >>> Requesting an initial pull. Thanks! >>> >>> -Wiz >>> >>> >>> ------------------------------------------------------------------------------ >>> The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE: >>> Pinpoint memory and threading errors before they happen. >>> Find and fix more than 250 security defects in the development cycle. >>> Locate bottlenecks in serial and parallel code that limit performance. >>> http://p.sf.net/sfu/intel-dev2devfeb >>> _______________________________________________ >>> Xbmc-addons mailing list >>> [email protected] >>> https://lists.sourceforge.net/lists/listinfo/xbmc-addons >>> >>> >> >> Thanks for the submission, looks very nice. >> >> 1. Please move the theme stuff to a folder called themes/ in the root >> dir. See alaska for an example. They must _not_ be xbt'd. This is the >> only way our packager will include the themes correctly. >> 2. skin.simplicity/720p/Startup.xml line 43: &alarm -> &alarm >> >> Nice work, I really don't have much else to complain about :) Once >> those are fixed up I'll pull. >> >> Cory >> >
Looks good, I'll pull aad9bcf. Thanks for fixing the version, I hadn't noticed. As a side-note, why do you delete/recreate files in separate commits? That defeats the purpose ;). It also makes review almost impossible. Cory ------------------------------------------------------------------------------ The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE: Pinpoint memory and threading errors before they happen. Find and fix more than 250 security defects in the development cycle. Locate bottlenecks in serial and parallel code that limit performance. http://p.sf.net/sfu/intel-dev2devfeb _______________________________________________ Xbmc-addons mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/xbmc-addons
