On 10/13/2014 11:19 PM, Ruben Undheim wrote: > I've already made Andreas upload one package for me today, but I > thought that since this package is a new one, I would spare Andreas > from some work. But if I tell him that you've reviewed it, it will > probably make his job a bit easier.
That's very considerate, but I don't expect any problems with small packages like these. > Regarding override_dh_auto_install: It is actually used. It prevents > the broken "make install" from running and lets sosi2osm.install take > care of which files should be installed. If I remove it, I will have > to create a patch that fixes the make install target. I could do that, > or I could add a comment to the rules file stating why it's there.. > Which one ? That's a good reason to keep it, adding a comment explaining that `make install` is explicitly skipped is a good idea. Kind Regards, Bas -- GPG Key ID: 4096R/E88D4AF1 Fingerprint: 8182 DE41 7056 408D 6146 50D1 6750 F10A E88D 4AF1 _______________________________________________ Pkg-grass-devel mailing list Pkg-grass-devel@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-grass-devel