[Bug 1344101] Review Request: rubygem-sinatra-cross_origin - Cross Origin Resource Sharing helper for Sinatra

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1344101



--- Comment #7 from František Dvořák <valtri@xxxxxxxxxx> ---
(In reply to Jun Aruga from comment #6)
> Hi, František Dvořák
> 
> I reviewed it. I want to ask you below points.
> 
> # Summary
> 
> ## 1.
> 
> > %if 0%{?rhel} && 0%{?rhel} <= 7
> > Requires:       ruby(rubygems)
> > Provides:       rubygem(%{gem_name}) = %{version}
> > %endif
> 
> Do you want to use this pacakge for rhel too now?
> Actually I am not confident that this 2 lines are correct for condition rhel
> <= 7. I can agree with your style if you are confident for the 2 lines.
> 

It is true these lines could be removed.

These lines were always needed for EPEL7 (and for F20, which is not needed to
check anymore), but it it true I'm not going to package it for EPEL7 right now.
I'm interested in EPEL7, but I must explore the sinatra package first, which is
not in EPEL7.

> ## 2.
> 
> > %files
> > ... 
> > %exclude %{gem_instdir}/%{gem_name}.gemspec
> > %exclude %{gem_instdir}/Rakefile
> > %exclude %{gem_instdir}/VERSION
> 
> In my style, I prefer those is included to %files doc section (doc-rpm).
> 
> My style is
> - *. gem_cache was excluded.
> - minimam files to run are included to %files.
> - Other text files are included to %files doc.
> because I like kind of same style with output of gem2rpm.
> 
> However I can agree with you style too, as it is gray area.
> 
> I want to respect your idea as much as possible if we have different style
> and it is not violation for the Guideline.
> 

Thanks. I don't have strong preferences here and to be close to gem2rpm means
to be more consistent with other packages. Updated.

> 
> ## 3.
> 
> > [?]: Package should compile and build into binary rpms on all supported
> >      architectures.
> 
> Could you show me URL of Koji scratch build?
> 

The new koji URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=15203098


Updated version:

Spec URL:
http://scientific.zcu.cz/fedora/rubygem-sinatra-cross_origin-0.3.2-2/rubygem-sinatra-cross_origin.spec
SRPM URL:
http://scientific.zcu.cz/fedora/rubygem-sinatra-cross_origin-0.3.2-2/rubygem-sinatra-cross_origin-0.3.2-2.fc26.src.rpm

%changelog
* Wed Aug 10 2016 František Dvořák <valtri@xxxxxxxxxx> - 0.3.2-2
- Update files section

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]