On Tue, 2007-01-23 at 08:56 +0100, Michael Schwendt wrote:
> On Sun, 7 Jan 2007 12:54:50 +0100, Michael Schwendt wrote:
>
> > On Sun, 7 Jan 2007 01:23:48 -0500, Anthony Green (green) wrote:
> >
> > > Author: green
> > >
> > > Update of /cvs/extras/rpms/ladspa-swh-plugins/FC-6
> > >
> > > Modified Files:
> > > ladspa-swh-plugins.spec
> > > Log Message:
> > > Add SELinux support.
> >
> > > +%post
> > > +# Set SELinux file context in the policy
> > > +semanage fcontext -f -- -a -t textrel_shlib_t '%{_libdir}/ladspa/sc4m_1916.so' 2>/dev/null || :
> > > +# Actually change the context
> > > +chcon -R -t textrel_shlib_t %{_libdir}/ladspa/sc4m_1916.so || :
> > > +
> > > +%postun
> > > +# SELinux support
> > > +if [ $1 -eq 0 ]; then # final removal
> > > + semanage fcontext -f -- -d -t textrel_shlib_t %{_libdir}/ladspa/sc4m_1916.so || :
> > > +fi
> > > +
> >
> > Shouldn't this be built with -fPIC/-fpic instead of changing the SELinux
> > context?
>
> +1/-1 from anyone else, please.
+1 unless there's some really good reason why that's not possible or it
wouldn't work.
Paul.
--
fedora-extras-list mailing list
fedora-extras-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-extras-list
