sclorg / spec2scl

MIT License
7 stars 7 forks source link

false positive replacement by spec2scl #2

Closed torsava closed 7 years ago

torsava commented 11 years ago

Originally reported by: Anonymous


I have in %files section:

%files %dir %{gem_instdir} %{gem_instdir}/app %{gem_instdir}/lib %exclude %{gem_cache} %{gem_spec} %doc %{gem_instdir}/MIT-LICENSE %doc %{gem_instdir}/APACHE-LICENSE-2.0

%exclude %{gem_instdir}/spec %exclude %{gem_instdir}/rel-eng %exclude %{gem_instdir}/.gitignore %exclude %{gem_instdir}/.rspec %exclude %{gem_instdir}/.travis.yml %exclude %{gem_dir}/cache/%{gem_name}-%{version}.gem

And spec2scl will do: %{?scl:scl enable %{scl} "} %exclude %{gem_instdir}/.rspec %{?scl:"}

Which I believe is incorrect because it rpmbuild will end up with: RPM build errors: File must begin with "/": scl File must begin with "/": enable File must begin with "/": ruby193 File must begin with "/": File must begin with "/":


torsava commented 11 years ago

Original comment by Slavek Kabrda (Bitbucket: bkabrda, GitHub: bkabrda):


Prevent .rspec from being treated as an invocation. Fixes #2 (but should be fixed generally by implementing #4).