On Thu, Nov 03, 2011 at 07:56:56PM -0700, Linus Arver wrote: > On Thu, Nov 03, 2011 at 08:25:27AM +0100, Lukas Fleischer wrote: > > On Wed, Nov 02, 2011 at 06:13:52PM -0700, Linus Arver wrote: > > > On Wed, Nov 02, 2011 at 08:59:15PM -0400, Dave Reisner wrote: > > > > On Wed, Nov 02, 2011 at 05:49:27PM -0700, Linus Arver wrote: > > > > > For all vcs prototypes, we currently create a temporary build > > > > > directory. > > > > > However, we do not delete it after we are done creating the package. > > > > > This wastes disk space. > > > > > > > > > > We already do a "rm -rf" on the build directories every time after we > > > > > do > > > > > a checkout anyway, so this patch doesn't really change anything. > > > > > > > > > > > > > Big -1 from me. This prevents repackaging. > > > > > > > > dave > > > > [...] > > > > > > I'm thinking of keeping this patch, but just as a comment for all the > > > thousands of AUR packagers who blindly use the prototype without taking > > > into account their particular needs. > > > > > > Any objections? > > > > Putting a comment there might encourage AUR packagers to uncomment that, > > especially if they don't know what they're doing. On the other side, > > users, who know how makepkg(1) works, also know how to clean/remove the > > build directory and don't need such a comment... My two cents. > > So you're suggesting > > # code > > instead of > > # comment > # code
No, I'm suggesting not to patch anything. Don't mention the `rm -rf` stuff - just keep it as it is right now. The comment will confuse novice packagers (and novice users who try to repackage stuff!) and won't help advanced packagers. > > ? Well, if that's the case then I must disagree --- I *despise* it when > there's code commented out with zero explanation. Looking back, I can > see that the comment I wrote is rather verbose; I'll shorten it to a > one-liner, for now, like this: > > # save disk space > # rm -rf /temp/build/dir > > Everyone loves one-liner comments, right? > > -Linus > > P.S. I'll wait and let the discussion die down for a few days this time > before releasing v3 of this patch series... the traffic for ABS > patches is nearly non-existant anyway.
