[Touch-packages] [Bug 2004592] Re: aalogparse.h cannot be included from C++ code

2023-02-04 Thread Steve Langasek
FWIW this is less interesting to me personally now, as I found the right
switches to abi-compliance-checker to deal with C++ keywords as C
variable names.  And also it turns out there are a lot of C headers that
have this problem.  So while it's probably still interesting to someone
to be able to use these headers from C++ code, that someone is no longer
me :)

** Changed in: apparmor (Ubuntu)
   Importance: Undecided => Wishlist

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apparmor in Ubuntu.
https://bugs.launchpad.net/bugs/2004592

Title:
  aalogparse.h cannot be included from C++ code

Status in apparmor package in Ubuntu:
  New

Bug description:
  aalogparse.h cannot be included from C++ code because it uses
  'namespace' and 'class' as variable names, and these are reserved
  keywords in C++.

  /usr/include/aalogparse/aalogparse.h:137:15: error: expected unqualified-id 
before ‘namespace’
137 | char *namespace;
|   ^
  /usr/include/aalogparse/aalogparse.h:163:15: error: expected unqualified-id 
before ‘class’
163 | char *class;
|   ^

  This is interesting to me because abi-compliance-checker necessarily
  processes all headers as C++; but it's likely of general interest to
  be able to link C++ code to libapparmor.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apparmor/+bug/2004592/+subscriptions


-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp


[Touch-packages] [Bug 2004592] Re: aalogparse.h cannot be included from C++ code

2023-02-04 Thread John Johansen
Sigh, so this one is a bit of a pita. Minimal C code uses this and it is
easy to update that. The problem comes with the SWIG interface libraries
(python, ruby, etc) have baked these names into their API. We can update
those but then we stand a good chance of breaking some external code.

Admittedly I don't know SWIG well, but I am looking into what can be
done.

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apparmor in Ubuntu.
https://bugs.launchpad.net/bugs/2004592

Title:
  aalogparse.h cannot be included from C++ code

Status in apparmor package in Ubuntu:
  New

Bug description:
  aalogparse.h cannot be included from C++ code because it uses
  'namespace' and 'class' as variable names, and these are reserved
  keywords in C++.

  /usr/include/aalogparse/aalogparse.h:137:15: error: expected unqualified-id 
before ‘namespace’
137 | char *namespace;
|   ^
  /usr/include/aalogparse/aalogparse.h:163:15: error: expected unqualified-id 
before ‘class’
163 | char *class;
|   ^

  This is interesting to me because abi-compliance-checker necessarily
  processes all headers as C++; but it's likely of general interest to
  be able to link C++ code to libapparmor.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apparmor/+bug/2004592/+subscriptions


-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp