All of lore.kernel.org
 help / color / mirror / Atom feed
* ANN: SELinux Userspace Release 20160107-rc1
@ 2016-01-07 15:56 Steve Lawrence
  2016-01-07 22:16 ` Nicolas Iooss
  2016-01-11 11:34 ` Petr Lautrbach
  0 siblings, 2 replies; 4+ messages in thread
From: Steve Lawrence @ 2016-01-07 15:56 UTC (permalink / raw)
  To: SELinux List

A release candidate for SELinux Userspace is now available at:

https://github.com/SELinuxProject/selinux/wiki/Releases

This has been tagged as 20160107-rc1 in the git repository.

Please give at a test and let us know if there are any issues.

Thanks,
- Steve

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: ANN: SELinux Userspace Release 20160107-rc1
  2016-01-07 15:56 ANN: SELinux Userspace Release 20160107-rc1 Steve Lawrence
@ 2016-01-07 22:16 ` Nicolas Iooss
  2016-01-08 14:30   ` Steve Lawrence
  2016-01-11 11:34 ` Petr Lautrbach
  1 sibling, 1 reply; 4+ messages in thread
From: Nicolas Iooss @ 2016-01-07 22:16 UTC (permalink / raw)
  To: selinux

On 01/07/2016 04:56 PM, Steve Lawrence wrote:
> A release candidate for SELinux Userspace is now available at:
> 
> https://github.com/SELinuxProject/selinux/wiki/Releases
> 
> This has been tagged as 20160107-rc1 in the git repository.
> 
> Please give at a test and let us know if there are any issues.
> 
> Thanks,
> - Steve

Hello,

As I am updating my packages to test this -rc1, I encountered three very
minor problems with secilc:

1. The README says "libsepol >= 2.4", which seems not to be true.  I get
this error when compiling secilc with libsepol 2.4:

  secilc.c:298:7: error: too few arguments to function
  ‘cil_filecons_to_string’
    rc = cil_filecons_to_string(db, &fc_buf, &fc_size);
         ^
  In file included from secilc.c:40:0:
  /usr/include/sepol/cil/cil.h:47:12: note: declared here
    extern int cil_filecons_to_string(cil_db_t *db, sepol_policydb_t
  *sepol_db, char **o
              ^
  <builtin>: recipe for target 'secilc.o' failed
  make: *** [secilc.o] Error 1

The build is fine with libsepol 2.5-rc1, so I guess the dependency only
needs to be updated in the README file.

2. "make" does not build the man page.  This is because it defaults to
the first defined target in the Makefile, which is secilc, not "all",
unlike the other SELinux projects.  I was a little bit surprised that
"make" did not behave like "make all" but I guess this is not a big
deal.  Feel free not to "fix" this.

3. While reading the Makefile, I found that "man" target was missing
from the .PHONY line [2] and I do not know the precise consequences of
an incomplete .PHONY target.  Anyway "make all && make install" works fine.

Thanks,
Nicolas

[1]
https://github.com/SELinuxProject/selinux/blob/secilc-2.5-rc1/secilc/README#L14
[2]
https://github.com/SELinuxProject/selinux/blob/secilc-2.5-rc1/secilc/Makefile#L49

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: ANN: SELinux Userspace Release 20160107-rc1
  2016-01-07 22:16 ` Nicolas Iooss
@ 2016-01-08 14:30   ` Steve Lawrence
  0 siblings, 0 replies; 4+ messages in thread
From: Steve Lawrence @ 2016-01-08 14:30 UTC (permalink / raw)
  To: Nicolas Iooss, selinux

On 01/07/2016 05:16 PM, Nicolas Iooss wrote:
> On 01/07/2016 04:56 PM, Steve Lawrence wrote:
>> A release candidate for SELinux Userspace is now available at:
>>
>> https://github.com/SELinuxProject/selinux/wiki/Releases
>>
>> This has been tagged as 20160107-rc1 in the git repository.
>>
>> Please give at a test and let us know if there are any issues.
>>
>> Thanks,
>> - Steve
> 
> Hello,
> 
> As I am updating my packages to test this -rc1, I encountered three very
> minor problems with secilc:
> 
> 1. The README says "libsepol >= 2.4", which seems not to be true.  I get
> this error when compiling secilc with libsepol 2.4:
> 
>   secilc.c:298:7: error: too few arguments to function
>   ‘cil_filecons_to_string’
>     rc = cil_filecons_to_string(db, &fc_buf, &fc_size);
>          ^
>   In file included from secilc.c:40:0:
>   /usr/include/sepol/cil/cil.h:47:12: note: declared here
>     extern int cil_filecons_to_string(cil_db_t *db, sepol_policydb_t
>   *sepol_db, char **o
>               ^
>   <builtin>: recipe for target 'secilc.o' failed
>   make: *** [secilc.o] Error 1
> 
> The build is fine with libsepol 2.5-rc1, so I guess the dependency only
> needs to be updated in the README file.
> 
> 2. "make" does not build the man page.  This is because it defaults to
> the first defined target in the Makefile, which is secilc, not "all",
> unlike the other SELinux projects.  I was a little bit surprised that
> "make" did not behave like "make all" but I guess this is not a big
> deal.  Feel free not to "fix" this.
> 
> 3. While reading the Makefile, I found that "man" target was missing
> from the .PHONY line [2] and I do not know the precise consequences of
> an incomplete .PHONY target.  Anyway "make all && make install" works fine.
> 
> Thanks,
> Nicolas
> 
> [1]
> https://github.com/SELinuxProject/selinux/blob/secilc-2.5-rc1/secilc/README#L14
> [2]
> https://github.com/SELinuxProject/selinux/blob/secilc-2.5-rc1/secilc/Makefile#L49
> 

Thanks for testing. You're correct on all counts. I'll send a patch to
fix these issues shortly.

Thanks,
- Steve

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: ANN: SELinux Userspace Release 20160107-rc1
  2016-01-07 15:56 ANN: SELinux Userspace Release 20160107-rc1 Steve Lawrence
  2016-01-07 22:16 ` Nicolas Iooss
@ 2016-01-11 11:34 ` Petr Lautrbach
  1 sibling, 0 replies; 4+ messages in thread
From: Petr Lautrbach @ 2016-01-11 11:34 UTC (permalink / raw)
  To: selinux

[-- Attachment #1: Type: text/plain, Size: 1046 bytes --]

On 01/07/2016 04:56 PM, Steve Lawrence wrote:
> A release candidate for SELinux Userspace is now available at:
> 
> https://github.com/SELinuxProject/selinux/wiki/Releases
> 
> This has been tagged as 20160107-rc1 in the git repository.
> 
> Please give at a test and let us know if there are any issues.
> 

I've built two testing set of packages for Fedora and both looks good so
far.


* http://copr.fedoraproject.org/coprs/plautrba/selinux-master/

It's built from SELinuxProject/selinux tree without any Fedora
modifications.


* http://copr.fedoraproject.org/coprs/plautrba/selinux/

Builds are based on 20160107-rc1 release with Fedora modifications
tracked in fedora-selinux/selinux repository. The modifications are
above all:

 * python scripts defaults to /usr/bin/python3
 * sepolicy is patched to be usable with python 3
 * other changes not-accepted by upstream

I'm going to push this one to Fedora Rawhide during this week.


Feel free to use and test both sets.

Petr
-- 
Petr Lautrbach



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-01-11 11:34 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-07 15:56 ANN: SELinux Userspace Release 20160107-rc1 Steve Lawrence
2016-01-07 22:16 ` Nicolas Iooss
2016-01-08 14:30   ` Steve Lawrence
2016-01-11 11:34 ` Petr Lautrbach

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.