[Bug 588428] Review Request: rubygem-addressable - Imporved URI/URL handling

2011-07-19 Thread bugzilla
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=588428

Shawn Starr  changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 CC||shawn.st...@rogers.com
 Resolution|DEFERRED|
   Keywords||Reopened

--- Comment #3 from Shawn Starr  2011-07-19 11:58:23 
EDT ---
I will take this over, needed for OpenNebula 3.0.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588428] Review Request: rubygem-addressable - Imporved URI/URL handling

2011-07-19 Thread bugzilla
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=588428

Shawn Starr  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|shawn.st...@rogers.com

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588428] Review Request: rubygem-addressable - Imporved URI/URL handling

2010-12-01 Thread bugzilla
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=588428

Jason Tibbitts  changed:

   What|Removed |Added

  Status Whiteboard|StalledSubmitter|

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588428] Review Request: rubygem-addressable - Imporved URI/URL handling

2010-11-18 Thread bugzilla
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=588428

Jesus M. Rodriguez  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||DEFERRED
Last Closed||2010-11-18 09:52:46

--- Comment #2 from Jesus M. Rodriguez  2010-11-18 09:52:46 
EST ---
This initially started as a dependency for rubygem-buildr. It is no longer
needed for that. We can cancel this review.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588428] Review Request: rubygem-addressable - Imporved URI/URL handling

2010-11-18 Thread bugzilla
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=588428

Jason Tibbitts  changed:

   What|Removed |Added

  Status Whiteboard||StalledSubmitter

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588428] Review Request: rubygem-addressable - Imporved URI/URL handling

2010-08-29 Thread bugzilla
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=588428

Michael Stahnke  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||mastah...@gmail.com

--- Comment #1 from Michael Stahnke  2010-08-29 22:16:51 
EDT ---
builds in mock
rpmlint is clean

%ruby_sitelib should be removed, as it is not used.


You could add a %check

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


There isn't really a reason to do this. You can simply mark the normally
installed README file a %doc.  
mkdir -p %{buildroot}/usr/share/doc/%{gemdir}-%{version}
mv  %{buildroot}%{geminstdir}/README
%{buildroot}/usr/share/doc/%{gemdir}-%{version}

I'd also encourage you to break this package into a separate -doc package. It
would allow someone wishing to use addressable as a dep to not install about
3/4 of the total files.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588428] Review Request: rubygem-addressable - Imporved URI/URL handling

2010-05-06 Thread bugzilla
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=588428

Adam Young  changed:

   What|Removed |Added

 Blocks||588406

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review