Bug 703178 - Review Request: rubygem-declarative_authorization - The declarative_authorization plug in provides readable auth rules for Rails.
Summary: Review Request: rubygem-declarative_authorization - The declarative_authoriza...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Daiki Ueno
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-05-09 15:06 UTC by Darryl L. Pierce
Modified: 2015-06-22 00:07 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-06-07 17:36:02 UTC
Type: ---
Embargoed:
dueno: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Darryl L. Pierce 2011-05-09 15:06:25 UTC
Spec URL: http://mcpierce.fedorapeople.org/rpms/rubygem-declarative_authorization.spec
SRPM URL: http://mcpierce.fedorapeople.org/rpms/rubygem-declarative_authorization-0.5.2-1.fc14.src.rpm
Description: The declarative_authorization plug in provides readable auth rules for Rails.

Comment 1 Daiki Ueno 2011-05-11 02:16:32 UTC
I'll take this as well as bug 703248 (actually I overlooked this in the NEW RR list).

It looks that the issues mentioned in bug 703248 were already addressed.  However, it fails to build:

http://koji.fedoraproject.org/koji/taskinfo?taskID=3064358

Comment 2 Darryl L. Pierce 2011-05-11 11:24:11 UTC
(In reply to comment #1)
> I'll take this as well as bug 703248 (actually I overlooked this in the NEW RR
> list).
> 
> It looks that the issues mentioned in bug 703248 were already addressed. 
> However, it fails to build:
> 
> http://koji.fedoraproject.org/koji/taskinfo?taskID=3064358

Maybe I'm misreading the log, but it looks like the build failure isn't due to the RPM or specfile but something on the build system itself:

+ cd /usr/lib/ruby/gems/1.8/gems/declarative_authorization-0.5.2
/var/tmp/rpm-tmp.K4xWdp: line 29: cd: /usr/lib/ruby/gems/1.8/gems/declarative_authorization-0.5.2: No such file or directory
RPM build errors:
error: Bad exit status from /var/tmp/rpm-tmp.K4xWdp (%check)
    Bad exit status from /var/tmp/rpm-tmp.K4xWdp (%check)
Child returncode was: 1
EXCEPTION: Command failed. See logs for output.
 # ['bash', '--login', '-c', 'rpmbuild -bb --target noarch --nodeps builddir/build/SPECS/rubygem-declarative_authorization.spec']
Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/mock/trace_decorator.py", line 70, in trace
    result = func(*args, **kw)
  File "/usr/lib/python2.6/site-packages/mock/util.py", line 325, in do
    raise mock.exception.Error, ("Command failed. See logs for output.\n # %s" % (command,), child.returncode)
Error: Command failed. See logs for output.
 # ['bash', '--login', '-c', 'rpmbuild -bb --target noarch --nodeps builddir/build/SPECS/rubygem-declarative_authorization.spec']
LEAVE do --> EXCEPTION RAISED

I can build the RPM both from spec and SRPM locally without a problem. So is this something I can fix or something that needs to be fixed in Koji?

Comment 3 Daiki Ueno 2011-05-12 02:53:19 UTC
(In reply to comment #2)

> + cd /usr/lib/ruby/gems/1.8/gems/declarative_authorization-0.5.2
> /var/tmp/rpm-tmp.K4xWdp: line 29: cd:
> /usr/lib/ruby/gems/1.8/gems/declarative_authorization-0.5.2: No such file or
> directory

It seems to be accessing a file outside of %buildroot.  I guess you need to prepend %{buildroot} to %{geminstdir} like:

 %check
 cd %{buildroot}%{geminstdir}
 rake test

However, even then it still fails to build on F-15:

 Using Rails from RubyGems (> 2.1.0)
 /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:53:in `gem_original_require': no such file to load -- initializer (LoadError)

Maybe the test suite cannot run with Rails 3 in standalone mode.  I think enabling test suite could be postponed until the upstream fixes this.

Comment 4 Darryl L. Pierce 2011-05-12 11:55:48 UTC
(In reply to comment #3)
> (In reply to comment #2)
> 
> > + cd /usr/lib/ruby/gems/1.8/gems/declarative_authorization-0.5.2
> > /var/tmp/rpm-tmp.K4xWdp: line 29: cd:
> > /usr/lib/ruby/gems/1.8/gems/declarative_authorization-0.5.2: No such file or
> > directory
> 
> It seems to be accessing a file outside of %buildroot.  I guess you need to
> prepend %{buildroot} to %{geminstdir} like:
> 
>  %check
>  cd %{buildroot}%{geminstdir}
>  rake test
> 
> However, even then it still fails to build on F-15:
> 
>  Using Rails from RubyGems (> 2.1.0)
>  /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:53:in
> `gem_original_require': no such file to load -- initializer (LoadError)
> 
> Maybe the test suite cannot run with Rails 3 in standalone mode.  I think
> enabling test suite could be postponed until the upstream fixes this.

I spoke with the upstream, Matthu Huggins, and he verified that the tests depend on Rails 3. I'm going to work with him to fix them up to run on F14/Rails 2.3.8 and F15/Rails 3 going forward.

Comment 5 Darryl L. Pierce 2011-05-13 16:39:00 UTC
(In reply to comment #3)
> Maybe the test suite cannot run with Rails 3 in standalone mode.  I think
> enabling test suite could be postponed until the upstream fixes this.

In that case, can we move forward with approving the package?

Comment 6 Darryl L. Pierce 2011-06-06 12:07:44 UTC
Ping reviewer - can we move forward with the package review, please?

Comment 7 Daiki Ueno 2011-06-07 00:55:51 UTC
Sorry for not getting back to you earlier.  The packaging looks fine except %check.
Also it would be nice to resolve "File listed twice" warning when building, as noted in bug 703248.

APPROVED.

Comment 8 Darryl L. Pierce 2011-06-07 14:07:08 UTC
New Package SCM Request
=======================
Package Name: rubygem-declarative_authorization
Short Description: The declarative_authorization plug in provides readable auth rules for Rails.
Owners: mcpierce
Branches: F15
InitialCC: mcpierce

Comment 9 Jason Tibbitts 2011-06-07 15:10:12 UTC
Git done (by process-git-requests).

Comment 10 Darryl L. Pierce 2011-06-07 17:36:02 UTC
Thanks for the quick repo creation, Jason!


Note You need to log in before you can comment on or make changes to this bug.