Package Details: deheader 1.6-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/
Keywords: c c++
Licenses: BSD
Provides: deheader
Submitter: anish
Maintainer: anish
Last Packager: anish
Votes: 9
Popularity: 0.000000
First Submitted: 2010-12-02 03:58
Last Updated: 2016-10-19 05:29

Latest Comments

« First ‹ Previous 1 2

anish commented on 2011-05-25 17:00

@trontonic weirdly with simply

-import sys, os, getopt, time, re, operator, commands
+import sys, os, getopt, time, re, operator

it is working on one of my machines. Just checked the other one, and I see the error again. I'd sent the patches to the author months ago, ESR is not interested. If I can't find a fix for this, I'll switch it back to python2. About the return 1, just habit :)

xyproto commented on 2011-05-25 10:09

Just installed and tested deheader. When running deheader in a directory containing .c-files, I get:

Traceback (most recent call last):
File "/usr/bin/deheader", line 1496, in <module>
remove, verbose))
File "/usr/bin/deheader", line 1416, in deheader
(st, t) = testcompile(sourcefile, maker, verbosity=max(1, verbose), showerrs=True)
File "/usr/bin/deheader", line 1353, in testcompile
(status, output) = commands.getstatusoutput(command)
NameError: global name 'commands' is not defined

The "commands" module is available in python2 but not in python3, so this is an issue with the python3 patches.

Just using the python2 version solves this problem and also has the bonus of leaving the problem of upgrading deheader to python3 to the developer(s) of deheader.
Sending the python3 patch to the deheader developer(s) would be the right course of action, IMO.

xyproto commented on 2011-05-25 09:58

Great, thanks for updating the package. Also, just to nitpick, "|| return 1" isn't needed anymore, ref: http://projects.archlinux.org/pacman.git/tree/NEWS. :)

anish commented on 2011-05-25 08:36

@trontonic Thanks for the suggestion. I'm going to stick with python 3, since that's the default on arch. Fixed to work with 3.2. Added in man pages and license change as suggested.

xyproto commented on 2011-05-20 09:55

Hi,

Thanks for submitting this package in the first place.

Unfortunately, the package did not work here, when running /usr/bin/deheader, I got:

Traceback (most recent call last):
File "/usr/bin/deheader", line 32, in <module>
import sys, os, getopt, time, re, operator, commands
ImportError: No module named commands

Also, gzip is not a needed dependency, patches should (preferably) be sent to the author, the license file should be installed, the man-page doesn't have to be gzipped manually, the package name should not be in the description and the installed files should have the right permissions (755 for executables and 644 for the rest). (ref: the arch wiki) "Namcap" can be a helpful utility.

Here's a working PKGBUILD: http://pastebin.com/wkGU70iP

xyproto commented on 2011-05-20 09:47

Here's a working PKGBUILD: http://pastebin.com/wkGU70iP

The package did not work here, when running /usr/bin/deheader, I got:
Traceback (most recent call last):
File "/usr/bin/deheader", line 32, in <module>
import sys, os, getopt, time, re, operator, commands
ImportError: No module named commands

anish commented on 2011-02-15 04:00

updated

anish commented on 2010-12-28 22:23

Updated to use python3. A python2 patch is still included in case anyone wants to stick with that. I'm no python expert, so please let me know if the patch causes any errors that I might have missed.

anish commented on 2010-12-28 21:32

@RedBeard0531 thanks. fixed & updated

Anonymous comment on 2010-12-28 06:14

You need updating for the python 2 -> 3 change