All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sven Vermeulen <sven.vermeulen@siphos.be>
To: selinux@tycho.nsa.gov
Subject: [PATCH/RFC] setools: update swig interfaces for swig-2.0.7 (and higher?)
Date: Mon, 20 Aug 2012 21:07:32 +0200	[thread overview]
Message-ID: <20120820190731.GA16363@siphos.be> (raw)

Hi guys,

In Gentoo, I had to patch the setools package' swig interfaces to build with
the swig-2.0.7 (and probably a few previous ones, version 2.0.4 still seemed
to build the code properly). The patch is fairly large (228 KiB) so I've put
it online for now. However, I'm not sure if this is the right way to fix
things or not.

In any case, the build failures we got were:

../qpol.i:241: Warning 504: Function qpol_module::qpol_module_t(char const
*) must have a return type. Ignored.
../qpol.i:244: Warning 521: Illegal destructor name
qpol_module::~qpol_module_t(). Ignored.

and

qpol_wrap.c:1831:24: error: conflicting types for qpol_module_get_path
../../../libqpol/include/qpol/module.h:69:13: note: previous declaration of qpol_module_get_path was here
qpol_wrap.c: In function qpol_module_get_path:
qpol_wrap.c:1834:3: error: too many arguments to function qpol_module_get_path
qpol_wrap.c:1831:24: note: declared here

I fixed it by changing the constructors/destructor names to the struct name
(as in "typedef struct qpol_module {} qpol_module_t" so the
constructur/destructor uses "qpol_module" rather than "qpol_module_t", and
by introducing %rename() calls for each collision found during the build
process.

The patch is at
http://dev.gentoo.org/~swift/patches/0008-fix_swig_build_failures_bug_430262.patch

If it is the right patch, I hope this saves a few hours of work for the
other developers ;-)

Wkr,
	Sven Vermeulen

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

             reply	other threads:[~2012-08-20 19:08 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-20 19:07 Sven Vermeulen [this message]
2012-09-05 17:37 ` [PATCH/RFC] setools: update swig interfaces for swig-2.0.7 (and higher?) Christopher J. PeBenito
  -- strict thread matches above, loose matches on Subject: below --
2012-08-21 16:15 Guido Trentalancia
2012-08-24 16:18 ` Sven Vermeulen
2012-08-24 17:17   ` Eric Paris
2012-08-25 16:47     ` Sven Vermeulen
2012-08-29 15:00       ` Christopher J. PeBenito

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120820190731.GA16363@siphos.be \
    --to=sven.vermeulen@siphos.be \
    --cc=selinux@tycho.nsa.gov \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.