Fixes #5223 - Fix migration of rudder-webapp to 2.11 on RPM #442
Conversation
| @@ -179,6 +179,10 @@ mkdir -p %{buildroot}/etc/%{apache_vhost_dir}/ | |||
| mkdir -p %{buildroot}/etc/sysconfig/ | |||
| mkdir -p %{buildroot}/usr/share/doc/rudder | |||
|
|
|||
| # Emulate installation of file rudder.xml in order to be owned by package | |||
| mkdir -p %{buildroot}%{rudderdir}/jetty7/contexts/ | |||
| touch %{buildroot}%{rudderdir}/jetty7/contexts/rudder.xml | |||
There was a problem hiding this comment.
I'm pretty sure we don't want to install an empty file.
My understanding of the %ghost directive is that you can just specify it, and RPM will "own" the file, but not install it. Since you don't install it, you don't need it.
Did you test this with just the "%ghost", and no file, and have problems? If so, please explain what they were.
There was a problem hiding this comment.
It is explained in the documentation (http://www.rpm.org/max-rpm-snapshot/s1-rpm-inside-files-list-directives.html) that we need to use a dummy file in the buildroot:
"The way to achieve this, is to use the %ghost directive. By adding this directive to the line containing a file, RPM will know about the ghosted file, but will not add it to the package. However it still needs to be in the buildroot."
…11_on_rpm Fixes #5223 - Fix migration of rudder-webapp to 2.11 on RPM
Fixes #5223 - Fix migration of rudder-webapp to 2.11 on RPM
cf http://www.rudder-project.org/redmine/issues/5223
The migration removed the file /opt/rudder/jetty7/contexts/rudder.xml since it was not owned by package.
I've built a package to try the patch and it worked