Quoting Leo Iannacone (2014-04-30 23:26:44) > On 30 April 2014 23:11, Jérémy Lal <kapo...@melix.org> wrote: > > It's not all right to remove other files without good reasons. > > Here, you can have a doubt about the license of the other css file, but > > there is no doubt the js file is correctly licensed, so no reason to > > prune it. > > But ... does it make sense leave snip of code that does not work? > > I mean, if you `apt-get source' the package you will see run > bechmark/index.js. You may want to exec it.. and then?? > It fails.. because try to open large.css, excluded from source. > > $ grep css benchmark/index.js > var small = fs.readFileSync('benchmark/small.css', 'utf8'); > var large = fs.readFileSync('benchmark/large.css', 'utf8') > > Is it not better in this case remove the whole directory?
The better approach is not to remove more code, but to complement the minimal code stripping with a patch that makes the remaining code work again. - Jonas -- * Jonas Smedegaard - idealist & Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ [x] quote me freely [ ] ask before reusing [ ] keep private
signature.asc
Description: signature
_______________________________________________ Pkg-javascript-devel mailing list Pkg-javascript-devel@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-javascript-devel