[Ovirt-devel] make ovirt-node-image.iso w/o its rpm

Perry Myers pmyers at redhat.com
Fri Oct 24 14:16:38 UTC 2008


Alan Pevec wrote:
> Dan Kenigsberg wrote:
>> Also, moving logic from .spec to Makefile is generally a good thing that
>> shortens dev cycles.
>> Please review the following patch and consider applying something on
>> these lines. I hope this is the place for such requests.
> 
> yes, that was on our TechnicalDebt TODO, thanks for doing it :)
>> +SELINUX_ENFORCING=$(shell getenforce)
> since we run build as non-root account, we need full path 
> /usr/sbin/getenforce
> 
>> +iso: repos.ks
>  +ovirt-node-image.iso: repos.ks
> 
> since we have known file output, it's better to use real target
> otherwise, iso should be listed in .PHONY
> 
>> +        Enforcing) setenforce Permissive ;; \
> need sudo /usr/sbin/setenforce Permissive
> 
> I'll git-send-email your patch now with those changes.

Alan or Dan, we should make these same changes in the spec/makefile.am for 
the ovirt-appliance.

Perry




More information about the ovirt-devel mailing list