Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=601160 --- Comment #5 from Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> 2010-06-10 14:08:34 EDT --- Well, * ruby(abi) dependency - must be strict equality (i.e. R: ruby(abi) = 1.8) * rubygem module related dependency - For rubygem module related dependency, please use "(Build)Requires: rubygem(foo)" style. ref: (althogh this for perl guideline:) https://fedoraproject.org/wiki/Packaging/Perl#Perl_Requires_and_Provides * Dependency for -doc subpackage - -doc subpackage should depend on main package for directory ownership issue (also see below) * Directory for non-arch-dependent ruby scripts - As I said, arch-dependent files (like C extentsion modules) must be moved to %ruby_sitearch. On the other hand, non-arch-dependent ruby scripts must not be moved from %geminstdir. * Movement of files under %_builddir - Currently build.log says in %check section: -------------------------------------------------------------- 199 + rake test 200 rm -f 201 touch ext/ruby-54f732744bed984d9d0d133997bf4128 202 cd ext 203 (in /builddir/build/BUILD/rubygem-rdiscount-1.6.3.2/usr/lib/ruby/gems/1.8/gems/rdiscount-1.6.3.2) 204 /usr/bin/ruby extconf.rb 205 checking for random()... yes 206 checking for srandom()... yes .... .... 211 cd ext && make clean && make 212 gcc -I. -I. -I/usr/lib/ruby/1.8/i386-linux -I. -DHAVE_RANDOM -DHAVE_SRANDOM -DHAVE_RAND -DHAVE_SRAND -D_FILE_OFFSET_BITS=64 -fPIC -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i686 -mtune=atom -fasynchronous-unwind-tables -fno-strict-aliasing -fPIC -c Csio.c .... 264 cp -p ext/rdiscount.so lib/ 265 /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in `gem_original_require': no such file to load -- markdown (LoadError) -------------------------------------------------------------- So currently "rake test" - tries to compile rdiscont.so again which was actually built in %build - and finally fails because lib/markdown.rb is not found (under %_builddir/%gemname-%version/%geminstdir). This is because rdiscount.so is moved to %ruby_sitearch before the tree is copied into %buildroot. You must - first copy the whole tree to %buildroot - then move files under %buildroot if needed. ! Note that as said above lib/markdown should not be moved to %ruby_sitearch. * "rake test" - For this gem "rake test" fails because the needed test/MarkdownTest_1.0.3 is missing. Please replace this with "rake test:unit" * Issues in %files - As %geminstdir is explicitly defined, please use the macro also in %files - The directory %geminstdir itself is not owned by any packages: https://fedoraproject.org/wiki/Packaging/UnownedDirectories#Common_Mistakes * %doc tag in -doc subpackage - Usually I think %doc attribute in -doc subpackage is just redundant because the name of the rpm already says that this is for documentation. * %exclude'ing ext/ directory - Leaves ext/rdiscount.so.debug files in -debuginfo rpm, which is not needed. Please "remove" ext/ directory from %buildroot completely (not from %_builddir) instead of using %exclude to avoid this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review