firebitsbr / skipfish

Automatically exported from code.google.com/p/skipfish
Apache License 2.0
0 stars 0 forks source link

Rename config.h -> skipfish.h #76

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
This is largely a religious issue ...

... but many are expecting AutoFu swaddled
projects where config.h is generated, not
included.

I've added all the Autofu to skipfish (if interested):
    cvs -d :ext:rpm-cvs@rpm5.org:/v/rpm/cvs co skipfish

Please note that if I had my druthers, I'd use your
pleasantly KISS and portable Makefile. Alas, I MUST
use AutoFu to embed skipfish @rpm5.org.

Meanwhile renaming config.h -> skipfish.h avoids
the accidental name collision and shouldn't hurt anything
whatsoever.

Original issue reported on code.google.com by n3npq....@gmail.com on 5 Jul 2010 at 2:15

GoogleCodeExporter commented 8 years ago
Actually, this would create the impression that skipfish.h contains function 
declarations for skipfish.c code, rather than configuration settings (which 
should be easy to locate).

I appreciate bug reports, but this seems... arbitrary :-) People are welcome to 
fork it off or apply distro-specific patches to rename files or directories, 
change versioning schemes, etc, but I'm not inclined to make this change 
upstream...

Original comment by lcam...@gmail.com on 5 Jul 2010 at 3:04

GoogleCodeExporter commented 8 years ago
Yep arbitrary == religious. But do look at issue #59, you're
soomed to choose some "make install" path shortly afaict and
that's where the AutoFu starts.

Again, your Makefile is quite pleasant ...

Original comment by n3npq....@gmail.com on 5 Jul 2010 at 3:14