Package Details: setools 3.3.8-5

Git Clone URL: https://aur.archlinux.org/setools.git (read-only)
Package Base: setools
Description: Policy analysis tools for SELinux
Upstream URL: https://github.com/TresysTechnology/setools3/wiki
Licenses: GPL, LGPL
Groups: selinux
Conflicts: selinux-setools
Provides: selinux-setools=3.3.8-5
Submitter: Siosm
Maintainer: IooNag
Last Packager: IooNag
Votes: 15
Popularity: 0.088816
First Submitted: 2013-11-03 20:05
Last Updated: 2015-12-09 12:47

Dependencies (18)

Required by (1)

Sources (14)

Latest Comments

IooNag commented on 2015-12-08 23:09

Hello. I did not change anything to the package, and you are right that swig is missing from makedepends. In fact I have it installed on my build system because it is in libselinux and libsemanage makedepends, but if you uninstall packages between builds, it no longer works. This is a bug that I will fix.

For the "The next patch would create the file apol/constraints_tab.tcl" error, it comes from the fact that makepkg is trying to apply a patch which has already been applied, in prepare(). You can either use "makepkg -C" to remove src/ before building or use "makepkg -e" to build the package from the already-extracted src/. Thanks!

Wabuo commented on 2015-12-08 22:17

To day it installed fine
thx

Wabuo commented on 2015-12-08 09:13

ERROR: A failure occurred in prepare().
Aborting...

The next patch would create the file apol/constraints_tab.tcl,
which already exists! Skipping patch.

Wabuo commented on 2015-12-08 09:03

missing dependencie= swig

IooNag commented on 2015-02-09 05:14

It's not a SWIG bug but an error in the patch. I reworked it and submitted upstream (https://github.com/TresysTechnology/setools3/pull/2). I'm waiting for upstream review/feedback before including the patch to the package, alongside with what's needed to support libsepol 2.4 (https://github.com/TresysTechnology/setools/commit/3937946900cf4f8bee2fdda697831b14ab8f849d).

fmorris commented on 2015-01-20 18:35

Ok, I managed to install the package removing the extra % from the Gentoo patch. Thanks! Should I still report the bug to SWIG?

IooNag commented on 2015-01-19 04:21

In fact, Gentoo no longer uses the patch but forces swig 1 to be installed (https://bugs.gentoo.org/show_bug.cgi?id=430262#c10 and http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/app-admin/setools/setools-3.3.8-r5.ebuild?view=markup ). As there are some backward-compatibility issues and I don't want to spend much time on it alone, I may update the package to use "swig1" or "swig2" AUR package if this works.

IooNag commented on 2015-01-19 03:21

Ok, I found what caused the issue. The Gentoo patch contains:
+%rename(seaudit_avc_message_get_message_type) %seaudit_avc_message_wrap_get_message_type;
... the second percent sign is invalid. Without it, this line works fine:
+%rename(seaudit_avc_message_get_message_type) seaudit_avc_message_wrap_get_message_type;
I'm going to report this bug against the Gentoo package (we are using Gentoo patch because upstream only support SWIG 2).

IooNag commented on 2015-01-19 02:54

I can reproduce your issue but I think it is a bug in SWIG, as I can build setools with swig 3.0.3-1. For a "quick & dirty fix", you can build an older version of SWIG by using this PKGBUILD: https://projects.archlinux.org/svntogit/packages.git/tree/trunk/PKGBUILD?h=packages/swig&id=c7b7029adf16fdc65bf336e8a119e928af5d997f

The Github issue you indicated tracks an other issue, with message "Assertion failed: (b->refcount > 0), function DohDelete, file DOH/base.c, line 42.", not "swig: DOH/base.c:36: DohDelete: Assertion `0' failed.". A quick look at the code (https://github.com/swig/swig/blob/rel-3.0.4/Source/DOH/base.c#L25 ) shows that a debug build of SWIG (with SWIG_DEBUG_DELETE) can show more verbose messages.

Could you please use a debug build of SWIG and report the messages you get to SWIG bugtracker?

fmorris commented on 2015-01-18 15:29

Building version 3.3.8-4 of the package fails with swig 3.0.4.

swig: DOH/base.c:36: DohDelete: Assertion `0' failed.
Makefile:688: recipe for target 'seaudit_wrap.c' failed
make[4]: *** [seaudit_wrap.c] Aborted (core dumped)

The swig error seems to be related to this issue
https://github.com/swig/swig/issues/280
which seems to have been fixed back on 2014-12-18... latest swig AUR build is 2015-01-15 so it should contain the fix?

All comments