Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-12-29 Thread Simon McVittie
On 29/12/14 07:19, Alexandre Detiste wrote: my patch still applies automatically. Thanks, I'll merge it. Here is a review; I'll probably fix some of this myself. You're mixing hard tabs (U+0009) and 4-space units for indentation. Please be consistent within each file, and consider configuring

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-12-29 Thread Simon McVittie
On 29/12/14 10:34, Simon McVittie wrote: +for file in $files; do +slipstream_file $file usr/share/games/tyrian/$file +done This will do all the work of repacking the .deb once for each of the 50ish files you're adding to it. Please ignore this bit, I was thinking of

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-12-29 Thread Alexandre Detiste
You're mixing hard tabs (U+0009) and 4-space units for indentation. Please be consistent within each file, and consider configuring your editor to display hard tabs. Ok, some of this dates back from april; I've since been bitten hard by this in some python project. Some of those dispredencies

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-12-28 Thread Alexandre Detiste
Hi, Nice to see some activity on GDP repository; my patch still applies automatically. https://github.com/a-detiste/game-data-packager BTW, I'd like to further help on this package. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble?

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-07-19 Thread Alexandre Detiste
Hi, Since april I learned to use git. You'll find here a repository with the patch: https://github.com/a-detiste/game-data-packager Diff: https://github.com/a-detiste/game-data-packager/commit/7a8f164b9c189584e0d5a9985ed41d774929346d Git Pull Url:

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-03-03 Thread Fabian Greffrath
Am Mittwoch, den 26.02.2014, 20:44 + schrieb Jonathan Dowland: Hi - what does this apply to, please? I tried against git master but not that. I guess the last package release? I had a brief attempt at cherry-picking the patch (which applied cleanly on top of 36) to master but it doesn't

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-02-26 Thread Jonathan Dowland
On Thu, Feb 20, 2014 at 09:52:14PM +0100, Alexandre Detiste wrote: Here is a new patch using new function gdp_unzip_paths(), this should work for everybody. I tested it with all three avaible unzippers. Hi - what does this apply to, please? I tried against git master but not that. I guess the

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-02-26 Thread Alexandre Detiste
This patch (v2), applies to the sources obtained with apt-get source game-data-packager on debian/testing; so currently v37. Here is a new patch based on git. Hi - what does this apply to, please? I tried against git master but not that. I guess the last package release? I had a brief attempt

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-02-24 Thread Etienne Millon
* Alexandre Detiste alexandre.deti...@gmail.com [140220 21:55]: Here is a new patch using new function gdp_unzip_paths(), this should work for everybody. I tested it with all three avaible unzippers. Hi, I tested the patch and it works great. Once I installed the package that it produced, I

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-02-23 Thread Jonathan Dowland
Thanks all for the patch and comments on the patch... I'll be taking a look at this over the next few days. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-02-20 Thread Alexandre Detiste
Hi, After digging in /usr/lib/game-data-packager/game-data-packager-shared , I found that gdp_unzip() is deprecated and when working archive with subfolders; his behaviour depends on installed packages (unzip versus 7zip). So it works for me because I have 7zip installed. Here is a new patch

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-02-19 Thread Alexandre Detiste
Package: game-data-packager Version: 37 Severity: wishlist Tags: patch Dear Maintainer, Please add support for Tyrian data using following patch. The data are freely usable, but not redistribuable. The binary will be provided by 'opentyrian' package. Here is the ITP bug for opentyrian:

Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-02-19 Thread Etienne Millon
Hi, After applying this patch I to run game-data-packager tyrian -w and got the following error: 2014-02-19 17:26:04 (1.10 MB/s) - '/tmp/game-data-packager.f61l7L/tyrian21.zip' saved [4754048/4754048] rm: cannot remove '*.exe': No such file or directory rmdir: failed to remove