#3 Remove backup file in %check and avoid unnecessary reversal
Opened 3 months ago by pvalena. Modified 3 months ago
rpms/ pvalena/rubygem-rspec-core warn-nonexistent  into  master

file modified
+1 -3

@@ -155,15 +155,13 @@ 

  # - and this causes some test errors on rspec-core.

  # fixing this error without modifying rspec-core "code" is very hard,

  # once modifying source itself.

- sed -i.warn lib/rspec/core/configuration_options.rb \

+ sed -i lib/rspec/core/configuration_options.rb \

  	-e '\@because the HOME environment variable is not set@s|RSpec\.warning|#RSpec.warning|'

  

  ruby -rrubygems -Ilib/ -S exe/rspec || \

  	ruby -rrubygems -Ilib/ -S exe/rspec --tag ~broken

  %endif

  

- mv lib/rspec/core/configuration_options.rb{.warn,}

- 

  %files

  %dir	%{gem_instdir}

  

(1) If %need_bootstrap_set is set then the file won't exist and build will fail due to following mv command.
(2) In %check everything is installed and there's no need to revert changes.