[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-22 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=502191


Hans de Goede  changed:

   What|Removed |Added

 Depends on||502189, 501323




-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-22 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=502191





--- Comment #1 from Bill Nottingham   2009-05-22 10:13:11 
EDT ---
We used to ship this a long time ago - why was it removed (and has whatever
caused that been fixed)?

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-22 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=502191


Jon Ciesla  changed:

   What|Removed |Added

 CC||l...@jcomserv.net




--- Comment #2 from Jon Ciesla   2009-05-22 10:21:26 EDT ---
That was my first thought.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-22 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=502191





--- Comment #3 from Hans de Goede   2009-05-22 10:24:05 
EDT ---
(In reply to comment #1)
> We used to ship this a long time ago - why was it removed (and has whatever
> caused that been fixed)?  

The issue was non free artwork and that has been fixed by upstream now
(in collaboration with Debian, its completely dfsg free now)

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-22 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=502191


Jon Ciesla  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|l...@jcomserv.net
   Flag||fedora-review?




--- Comment #4 from Jon Ciesla   2009-05-22 10:29:15 EDT ---
Cool.  I'll have a go.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-22 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=502191





--- Comment #5 from Jon Ciesla   2009-05-22 12:03:10 EDT ---
rpmlint clean.

URL should be http://chromium-bsu.sourceforge.net/

Rename README.license to chromium-README.license (but keep it README.license
when installed) to avoid SRPM clobbering.

Can't mock-test the BRs since ftgl isn't updated yet and libglpng isn't in
rawhide yet, but I see that review's underway.

Otherwise on full review, it looks great.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-22 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=502191





--- Comment #6 from Hans de Goede   2009-05-22 14:40:06 
EDT ---
Thanks for the review!

Here is an updated package with those 2 issues fixed:
Spec URL: http://people.atrpms.net/~hdegoede/chromium-bsu.spec
SRPM URL: http://people.atrpms.net/~hdegoede/chromium-bsu-0.9.14-2.fc11.src.rpm

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-23 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=502191


Bug 502191 depends on bug 502189, which changed state.

Bug 502189 Summary: Review Request: libglpng - Toolkit for loading PNG images 
as OpenGL textures
https://bugzilla.redhat.com/show_bug.cgi?id=502189

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-26 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=502191





--- Comment #7 from Jon Ciesla   2009-05-26 13:34:10 EDT ---
Good deal, still waiting on libglpng to show up in my mock buildroot. . .

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-26 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=502191





--- Comment #8 from Hans de Goede   2009-05-26 13:43:38 
EDT ---
(In reply to comment #7)
> Good deal, still waiting on libglpng to show up in my mock buildroot. . .  

Hmm, are you pointing mock at rawhide + F-11 updates ? rawhide == F-11 GA, so
get the latest and greatest you need rawhide + F-11 updates.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-26 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=502191





--- Comment #9 from Jon Ciesla   2009-05-26 15:30:22 EDT ---
[bu...@fawkes ~]$ cat /etc/mock/fedora-rawhide-i586.cfg
config_opts['root'] = 'fedora-rawhide-i586'
config_opts['target_arch'] = 'i586'
config_opts['chroot_setup_cmd'] = 'groupinstall buildsys-build'
config_opts['dist'] = 'fc12'  # only useful for --resultdir variable subst

config_opts['yum.conf'] = """
[main]
cachedir=/var/cache/yum
debuglevel=1
reposdir=/dev/null
logfile=/var/log/yum.log
retries=20
obsoletes=1
gpgcheck=0
assumeyes=1

# repos

[fedora]
name=fedora
mirrorlist=http://mirrors.fedoraproject.org/mirrorlist?repo=rawhide&arch=i586
failovermethod=priority

[local]
name=local
baseurl=http://koji.fedoraproject.org/static-repos/dist-rawhide-current/i386
cost=2000
enabled=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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-27 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=502191





--- Comment #10 from Hans de Goede   2009-05-27 03:24:06 
EDT ---
Hmm, ok so no F-11 updates not F-11 updates-testing in there. Note that even if
you did have that it still wont build as you also need the ftgl from F-12.

So I guess its best to wait till rawhide unfreezes from F-11 and becomes F-12.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-27 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=502191





--- Comment #11 from Mamoru Tasaka   2009-05-27 
05:57:26 EDT ---
By the way mock build fails even for dist-f12:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1379204

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-27 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=502191





--- Comment #12 from Hans de Goede   2009-05-27 07:21:46 
EDT ---
(In reply to comment #11)
> By the way mock build fails even for dist-f12:
> http://koji.fedoraproject.org/koji/taskinfo?taskID=1379204  

Woops, my bad, here is a fixed version:
Spec URL: http://people.atrpms.net/~hdegoede/chromium-bsu.spec
SRPM URL: http://people.atrpms.net/~hdegoede/chromium-bsu-0.9.14-3.fc11.src.rpm

And here is a working koji scratch build of it:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1379288

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-27 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=502191


Jon Ciesla  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #13 from Jon Ciesla   2009-05-27 08:33:28 EDT ---
Looks good to me.

APPROVED.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-27 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=502191


Hans de Goede  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #14 from Hans de Goede   2009-05-27 08:49:58 
EDT ---
(In reply to comment #13)
> Looks good to me.
> 
> APPROVED.  

Thanks!

New Package CVS Request
===
Package Name:  chromium-bsu
Short Description: Fast paced, arcade-style, top-scrolling space shooter
Owners:jwrdegoede
Branches:  F-11
InitialCC:

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-27 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=502191


Jason Tibbitts  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #15 from Jason Tibbitts   2009-05-27 11:25:01 
EDT ---
CVS done.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-05-27 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=502191


Hans de Goede  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Comment #16 from Hans de Goede   2009-05-27 13:31:18 
EDT ---
(In reply to comment #15)
> CVS done.  

Thanks, imported and build, closing.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502191] Review Request: chromium-bsu - Fast paced, arcade-style, top-scrolling space shooter

2009-06-02 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=502191


Bug 502191 depends on bug 501323, which changed state.

Bug 501323 Summary: Please update ftgl to 2.1.3rc5
https://bugzilla.redhat.com/show_bug.cgi?id=501323

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review