Package Details: deheader 1.3-1

Git Clone URL: https://aur.archlinux.org/deheader.git (read-only)
Package Base: deheader
Description: C and C++ file analyzer to determine which header enclusions can be removed while still allowing them to compile
Upstream URL: http://www.catb.org/esr/deheader/
Licenses: BSD
Provides: deheader
Submitter: anish
Maintainer: anish
Last Packager: anish
Votes: 9
Popularity: 0.000012
First Submitted: 2010-12-02 03:58
Last Updated: 2016-02-18 07:47

Dependencies (1)

Required by (0)

Sources (3)

Latest Comments

anish commented on 2013-02-27 21:14

@esr Sent

Anonymous comment on 2013-02-25 23:02

I see a reference to deaheader patches sent to me, but I have not seen them. Please resent to esr@thyrsus.com

anish commented on 2011-06-01 19:46

Like I said, I'm only using python3 because it's the default on arch, not a fan of forcing people to install a whole new version of python for running a 20K program. That said, the PKGBUILD has always had the option for people to choose python2 instead

xyproto commented on 2011-06-01 09:16

@anish

I'm testing deheader against a utility I use at work, which is a single C file. It does include <string.h>, and always has, so your conclusion about it not being present is wrong. It was your broken version of deheader that gave the false report, indicating it was not.

Yes, namcap is great for improving packages, but it's not always correct. Coreutils is one such incident. The 2to3 tool is another example of a tool that can not always be trusted.

When that's said, the latest patched version of deheader does seem to work here now, but I still think it's a bad idea to patch it, instead of just using the official version and depend on python2.

anish commented on 2011-06-01 05:10

@trontonic What code are you testing against ? I can't find any differences and I tried comparing some client-server tcp code, kernel module code & a network simulator. This might be an issue with the code that you're running against. Updated with the 2to3 conversion tool that ships with python.

coreutils was a namcap suggestion, as you suggested.

Plus,
deheader: in ./main.c, strcpy() portability requires <string.h>. is sorta obvious. If you're using string functions, you SHOULD be including string.h, I'd say the patched version is giving you the correct output.

xyproto commented on 2011-05-30 13:50

This package still does not work properly.

xyproto commented on 2011-05-26 16:31

Also, the package should not depend on coreutils.

xyproto commented on 2011-05-26 09:52

@anish, no it still does not work. When running your PKGBUILD, deheader gives a false report.

Here's the output for your patched version on a sample project:

deheader: in ./main.c, access() portability requires <unistd.h>.
deheader: in ./main.c, strcpy() portability requires <string.h>.
deheader: in ./main.c, popen() portability requires <stdio.h>.
deheader: in ./main.c, system() portability requires <stdlib.h>.
deheader: in ./main.c, tolower() portability requires <ctype.h>.
deheader: saw 1 files, 7 includes, 0 removable

And here's the output from the version produced by my working PKGBUILD, posted 6 days ago:

deheader: saw 1 files, 7 includes, 0 removable

Patches should be sent upstream instead of being included in the PKGBUILD.

anish commented on 2011-05-26 05:57

@trontonic I think I fixed the issue, can you check if it's working for you now ?

xyproto commented on 2011-05-25 23:15

Hey, we can make a deheader fork named decapitator, put it on github and upgrade it to python3. :)

All comments