kthakore / pangzero

Porting Pang Zero to new SDL API for Debian Release
GNU General Public License v2.0
12 stars 4 forks source link

Various fixes for pangzero #2

Closed jwrdegoede closed 12 years ago

jwrdegoede commented 12 years ago

Hi,

Short intro I'm the Fedora packager of pangzero, and since Fedora has moved to the latest perl-sdl, I've been working on upgrading my packages to your git version of pangzero. During testing I've found and fixed quite a few issues, see the patches in this git pull request.

Known remaining issues:

Thanks & Regards,

Hans

kthakore commented 12 years ago

Thank you! I will update this ASAP!

Kartik Thakore

On 2012-10-29, at 4:57 PM, Hans de Goede notifications@github.com wrote:

Hi,

Short intro I'm the Fedora packager of pangzero, and since Fedora has moved to the latest perl-sdl, I've been working on upgrading my packages to your git version of pangzero. During testing I've found and fixed quite a few issues, see the patches in this git pull request.

Known remaining issues:

Thanks & Regards,

Hans

You can merge this Pull Request by running:

git pull https://github.com/jwrdegoede/pangzero master

Or view, comment on, or merge it at:

https://github.com/kthakore/pangzero/pull/2 Commit Summary

File Changes

Patch Links

jwrdegoede commented 12 years ago

Hi,

Me again :) I've done some more work on pangzero fixing all remaining issues I'm aware of!

Regards,

Hans

kthakore commented 12 years ago

Awesome.

Can you run perl Build test? And check if that is working?

On Sat, Nov 3, 2012 at 12:17 PM, Hans de Goede notifications@github.comwrote:

Hi,

Me again :) I've done some more work on pangzero fixing all remaining issues I'm aware of!

Regards,

Hans

— Reply to this email directly or view it on GitHubhttps://github.com/kthakore/pangzero/pull/2#issuecomment-10040949.

jwrdegoede commented 12 years ago

Hi,

That seems to work fine too:

[hans@shalem pangzero]$ perl Build test t/00-run.t .. ok All tests successful. Files=1, Tests=1, 53 wallclock secs ( 0.02 usr 0.00 sys + 2.70 cusr 0.13 csys = 2.85 CPU) Result: PASS

Regards,

Hans

On 11/03/2012 07:18 PM, Kartik Thakore wrote:

Awesome.

Can you run perl Build test? And check if that is working?

On Sat, Nov 3, 2012 at 12:17 PM, Hans de Goede notifications@github.comwrote:

Hi,

Me again :) I've done some more work on pangzero fixing all remaining issues I'm aware of!

Regards,

Hans

— Reply to this email directly or view it on GitHubhttps://github.com/kthakore/pangzero/pull/2#issuecomment-10040949.

— Reply to this email directly or view it on GitHub https://github.com/kthakore/pangzero/pull/2#issuecomment-10042356.

kthakore commented 12 years ago

Awesome pulling now.

On Sat, Nov 3, 2012 at 3:06 PM, Hans de Goede notifications@github.comwrote:

Hi,

That seems to work fine too:

[hans@shalem pangzero]$ perl Build test t/00-run.t .. ok All tests successful. Files=1, Tests=1, 53 wallclock secs ( 0.02 usr 0.00 sys + 2.70 cusr 0.13 csys = 2.85 CPU) Result: PASS

Regards,

Hans

On 11/03/2012 07:18 PM, Kartik Thakore wrote:

Awesome.

Can you run perl Build test? And check if that is working?

On Sat, Nov 3, 2012 at 12:17 PM, Hans de Goede <notifications@github.com wrote:

Hi,

Me again :) I've done some more work on pangzero fixing all remaining issues I'm aware of!

Regards,

Hans

— Reply to this email directly or view it on GitHub< https://github.com/kthakore/pangzero/pull/2#issuecomment-10040949>.

— Reply to this email directly or view it on GitHub < https://github.com/kthakore/pangzero/pull/2#issuecomment-10042356>.

— Reply to this email directly or view it on GitHubhttps://github.com/kthakore/pangzero/pull/2#issuecomment-10042909.