Spec URL: https://mtasaka.fedorapeople.org/Review_request/ruby-gnome2-suite/rubygem-webkit2-gtk.spec SRPM URL: https://mtasaka.fedorapeople.org/Review_request/ruby-gnome2-suite/rubygem-webkit2-gtk-3.1.3-1.fc.src.rpm Description: Ruby/WebKit2GTK is a Ruby binding of WebKit2GTK+. Fedora Account System Username: mtasaka Koji scratch build F-27: https://koji.fedoraproject.org/koji/taskinfo?taskID=19406281 F-26: https://koji.fedoraproject.org/koji/taskinfo?taskID=19406283
Nice packaging. * I guess the Rakefile modification is not strictly needed, but it is there as prevention, because pkgconfig is not used? * Not using BR pkgconfig(webkit2gtk-4.0) is good idea, development dependency of webkitgtk4 is not needed here. * run-test.rb probably doesn't need to load modules (only the code at the end is needed). But it is actually good to perform only minimal changes as used here. * Usage of Obsoletes is well commented. Package APPROVED!
Thank you! (In reply to František Dvořák from comment #1) > Nice packaging. > > * I guess the Rakefile modification is not strictly needed, but it is there > as prevention, because pkgconfig is not used? This is because unless modifying Rakefile, pkgconfig(webkit2gtk-4.0) (i.e. webkitgtk4-devel) will be needed as BR, while actually webkitgtk4 is needed as BR (and Requires) (and -devel is not needed). > * Not using BR pkgconfig(webkit2gtk-4.0) is good idea, development > dependency of webkitgtk4 is not needed here. > > * run-test.rb probably doesn't need to load modules (only the code at the > end is needed). But it is actually good to perform only minimal changes as > used here. Note that "-extension RANDR" means _not_ loading RANDR, there was a bug that loading RANDR extension on Xvfb make it crash (current default is loading also RANDR extension), but I guess this bug is already fixed, will remove. > * Usage of Obsoletes is well commented. > > > Package APPROVED! Thank you. I will surely review your package.
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/rubygem-webkit2-gtk
Rebuilt on F-27, F-26, push request submitted on F-26. Thank you for review!!