xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.


Changes between Initial Version and Version 1 of Ticket #606, comment 2


Ignore:
Timestamp:
07/31/14 12:13:20 (7 years ago)
Author:
Antoine Martin
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #606, comment 2

    initial v1  
    44* In regards to #603, I agree the workaround should be removed.
    55
    6 * I removed the chcon workarounds because xpra works on EL6 and Fedora 20 with selinux without the workaround. I am not an expert, but it would appear that setting type texrel_shlib_t is not preferred (1) (2). I don't understand text relocation, so I have no objection if this is kept. A comment explaining why this workaround is in place would be useful, though.
     6* I removed the chcon workarounds because xpra works on EL6 and Fedora 20 with selinux without the workaround. I am not an expert, but it would appear that setting type {{{texrel_shlib_t}}} is not preferred (1) (2). I don't understand text relocation, so I have no objection if this is kept. A comment explaining why this workaround is in place would be useful, though.
    77
    8 * Regarding %debug_package, I removed it since debug packages get built automatically with mock. I have no strong feelings here, though if you decided to keep it, I think the line should go towards the top of the SPEC file. Note, most packages do not define %debug_package.
     8* Regarding {{{%debug_package}}}, I removed it since debug packages get built automatically with mock. I have no strong feelings here, though if you decided to keep it, I think the line should go towards the top of the SPEC file. Note, most packages do not define %debug_package.
    99
    1010(1) http://danwalsh.livejournal.com/2006/05/11/ (see execmod)