Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Jonathan Nieder
Hi, Erik Faye-Lund wrote: > On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras > wrote: >> Erik Faye-Lund wrote: >>> Please don't. This script is useful to build with the MSVC IDE, which >>> enables us to use their excellent debugger. >> >> If you want this script to remain in contrib, please: >>

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 12:57 PM, Felipe Contreras wrote: > Erik Faye-Lund wrote: >> On Fri, May 9, 2014 at 11:32 AM, Felipe Contreras >> wrote: >> > Erik Faye-Lund wrote: >> >> On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras >> >> > You think changing the execution bit of a file is considered "

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread David Kastrup
Felipe Contreras writes: > David Kastrup wrote: >> Felipe Contreras writes: >> >> > David Kastrup wrote: >> > >> >> The idea of removing software from distribution is to get rid of >> >> stuff without a user base rather than punishing lazy developers. >> > >> > No. >> >> So we have you on reco

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
Erik Faye-Lund wrote: > On Fri, May 9, 2014 at 11:32 AM, Felipe Contreras > wrote: > > Erik Faye-Lund wrote: > >> On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras > >> > You think changing the execution bit of a file is considered "activity"? > >> > >> Well, now we're getting into semantics, whic

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
David Kastrup wrote: > Felipe Contreras writes: > > > David Kastrup wrote: > > > >> The idea of removing software from distribution is to get rid of > >> stuff without a user base rather than punishing lazy developers. > > > > No. > > So we have you on record that you would want to get rid of st

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 11:32 AM, Felipe Contreras wrote: > Erik Faye-Lund wrote: >> On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras >> wrote: >> > Erik Faye-Lund wrote: >> >> On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras >> >> wrote: >> >> > If you want this script to remain in contrib, pl

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread David Kastrup
Felipe Contreras writes: > David Kastrup wrote: > >> The idea of removing software from distribution is to get rid of >> stuff without a user base rather than punishing lazy developers. > > No. So we have you on record that you would want to get rid of stuff _with_ a user base and/or to punish l

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
David Kastrup wrote: > Felipe Contreras writes: > >> Besides, you say "No activity since 2010" - this is not the case, > >> bc380fc is from November 2013. > > > > You think changing the execution bit of a file is considered "activity"? > > For an indication of "this software has actual users", it

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
Erik Faye-Lund wrote: > On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras > wrote: > > Erik Faye-Lund wrote: > >> On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras > >> wrote: > >> > If you want this script to remain in contrib, please: > >> > > >> > a) Write at least a few tests > >> > b) Writ

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread David Kastrup
Felipe Contreras writes: > Erik Faye-Lund wrote: >> On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras >> wrote: >> > If you want this script to remain in contrib, please: >> > >> > a) Write at least a few tests >> > b) Write some documentation >> > c) Explain why it cannot live outside the gi

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras wrote: > Erik Faye-Lund wrote: >> On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras >> wrote: >> > If you want this script to remain in contrib, please: >> > >> > a) Write at least a few tests >> > b) Write some documentation >> > c) Explain wh

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
Erik Faye-Lund wrote: > On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras > wrote: > > If you want this script to remain in contrib, please: > > > > a) Write at least a few tests > > b) Write some documentation > > c) Explain why it cannot live outside the git.git repository like other > >

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras wrote: > Erik Faye-Lund wrote: >> On Fri, May 9, 2014 at 2:58 AM, Felipe Contreras >> wrote: >> > No activity since 2010, no documentation, no tests. >> > >> > Signed-off-by: Felipe Contreras >> > --- >> > contrib/buildsystems/Generators.pm

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
Erik Faye-Lund wrote: > On Fri, May 9, 2014 at 2:58 AM, Felipe Contreras > wrote: > > No activity since 2010, no documentation, no tests. > > > > Signed-off-by: Felipe Contreras > > --- > > contrib/buildsystems/Generators.pm| 42 -- > > contrib/buildsystems/Generators/QMake.pm | 189 --

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 2:58 AM, Felipe Contreras wrote: > No activity since 2010, no documentation, no tests. > > Signed-off-by: Felipe Contreras > --- > contrib/buildsystems/Generators.pm| 42 -- > contrib/buildsystems/Generators/QMake.pm | 189 - > contrib/buildsystems/Genera

[PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-08 Thread Felipe Contreras
No activity since 2010, no documentation, no tests. Signed-off-by: Felipe Contreras --- contrib/buildsystems/Generators.pm| 42 -- contrib/buildsystems/Generators/QMake.pm | 189 - contrib/buildsystems/Generators/Vcproj.pm | 626 -- contrib/buildsyst