Re: RFS: urbanterror-data

2008-03-25 Thread Gonéri Le Bouder
On Tue, Mar 25, 2008 at 04:34:23PM +1100, Jack Coulter wrote:

Hi Jack,

(...)
 Is there someway I can get a more detail description of what these 
 warnings mean so I can fix them?
You can call lintian with --info flag.
 
 Also, as far as I'm aware /var/cache/urbanterror-data is already removed 
 after successful package installation and configuration.
I removed the -data package with dpkg --purge because I didn't have enough
free space and in this case the /var/cache/urbanterror-data was kept.

I'm not fan of the -data package because it needs a lot of free space
and for most of the user it will be long to install. During this
process, all the apt-get upgrade a blocked.

I wonder if it's not a better solution to provide a script in the
urbanterror package that download urbanterror files. For example a
wrapper for the urbanterror command that check for the data files and
suggest to download the file if needed.

Cheers,

Gonéri



Re: RFS: urbanterror-data

2008-03-25 Thread Chris Lamb
Goneri Le Bouder wrote:

 I wonder if it's not a better solution to provide a script in the
 urbanterror package that download urbanterror files.  

Please don't do this. :(

 * There is plently of prior art in including large -data packages for
   games (nexuiz-data, for example).

 * It's unfair on people with low bandwidth or no internet access,
   especially as it won't even appear on Debian DVDs.

 * It bypasses existing mirroring setups, such as local mirrors or APT
   caches: If I host a LAN party, my local Debian mirror can provide most
   of the games we want to play at ~50MiB/s. However, if a game package was
   merely a wrapper, my guests would then start to--perhaps concurrently--
   downloading large files from the internet at a relatively awful speed.

   It would, of course, be possible to provide a means of supplying the
   file manually instead of downloading, but this is really awkward, and
   is no better than just providing the .deb of the -data package.


Regards,

-- 
Chris Lamb, UK   [EMAIL PROTECTED]
GPG: 0x634F9A20


signature.asc
Description: PGP signature


Re: RFS: urbanterror-data

2008-03-25 Thread Paul Wise
On Wed, Mar 26, 2008 at 9:46 AM, Chris Lamb [EMAIL PROTECTED] wrote:
 Goneri Le Bouder wrote:

   I wonder if it's not a better solution to provide a script in the
   urbanterror package that download urbanterror files.

  Please don't do this. :(

The ftpmasters (specifically Ganneff) suggested it be done with a
download script. Mainly this was because it is non-free anyway.

700 Mb .deb plus 700 Mb duplicated in the orig.tar.gz isn't acceptable
IMO. We need a way to have a .deb in the archive without an
orig.tar.gz before we can have such hge packages.

-- 
bye,
pabs

http://wiki.debian.org/PaulWise


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Re: RFS: urbanterror-data

2008-03-25 Thread Chris Lamb
Paul Wise wrote:

 The ftpmasters (specifically Ganneff) suggested it be done with a
 download script. Mainly this was because it is non-free anyway.
[...]
 700 Mb .deb plus 700 Mb duplicated in the orig.tar.gz isn't acceptable

Ah, I had assumed it was free software and about the ~100MB mark. I agree
with you.


Regards,

-- 
Chris Lamb, UK   [EMAIL PROTECTED]
GPG: 0x634F9A20


signature.asc
Description: PGP signature


RFS: urbanterror-data

2008-03-23 Thread Jack Coulter

Dear mentors,

I am still looking for a sponsor for my package urbanterror-data, the 
latest uploaded version has a number of fixes from previous revisions. I 
would greatly appreciate it if a sponsor could evaluate this package.


* Package name: urbanterror-data
  Version : 4.1
  Upstream Author : FrozenSand LLC [EMAIL PROTECTED]
* URL : http://www.urbanterror.net
* License : Quake 3 SDK EULA/Various free-to-distribute licenses
  Section : non-free/games

It builds these binary packages:
urbanterror-data - Urban Terror Data

The package appears to be lintian clean.

The upload would fix these bugs: 468325

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/non-free/u/urbanterror-data
- Source repository: deb-src http://mentors.debian.net/debian unstable
main contrib non-free
- dget
http://mentors.debian.net/debian/pool/non-free/u/urbanterror-data/urbanterror-data_4.1.dsc

I would be glad if someone uploaded this package for me.

Kind regards
 Jack Coulter



--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Re: RFS: urbanterror-data

2008-03-23 Thread Gonéri Le Bouder
On Sun, Mar 23, 2008 at 05:05:11PM +1100, Jack Coulter wrote:

Hi Jack,

Some remarks,

I don't understand the goal of the binary-common and why you fork a make
command. I did this change on your makefile:

--- ../save/urbanterror-data-4.1/debian/rules   2008-03-15 05:14:37.0 
+0100
+++ debian/rules2008-03-23 14:10:40.0 +0100
@@ -26,7 +26,9 @@
### Data files
dh_install -i

-binary-common:
+
+# Build architecture independant packages using the common target.
+binary-indep: install-indep
dh_testdir -i
dh_testroot -i
dh_installchangelogs -i
@@ -38,11 +40,6 @@
dh_md5sums -i
dh_builddeb -i

-
-# Build architecture independant packages using the common target.
-binary-indep: install-indep
-   $(MAKE) -f debian/rules DH_OPTIONS=-i binary-common
-
 # Build architecture dependant packages using the common target.
 binary-arch:
# Nothing to do

IMO, Pre-Depends can be avoid in you case, you should use Depends: here instead.

 Pre-Depends should be used sparingly, preferably only by packages whose 
premature
  upgrade or installation would hamper the ability of the system to continue 
with
  any upgrade that might be in progress.
http://www.debian.org/doc/debian-policy/ch-relationships.html

There is still some lintian warning:
W: urbanterror-data: no-debconf-config
W: urbanterror-data: malformed-title-in-templates urbanterror-data/license
W: urbanterror-data: too-long-extended-description-in-templates 
urbanterror-data/license
W: urbanterror-data: postinst-uses-db-input
W: urbanterror-data: missing-debconf-dependency
W: urbanterror-data: latest-debian-changelog-entry-without-new-version

Cheers,

Gonéri


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Re: RFS: urbanterror-data

2008-03-23 Thread Gonéri Le Bouder
On Sun, Mar 23, 2008 at 05:30:10PM +0100, Gonéri Le Bouder wrote:
 On Sun, Mar 23, 2008 at 05:05:11PM +1100, Jack Coulter wrote:
 
 Hi Jack,
 
 Some remarks,
Another points, the /var/cache/urbanterror-data should be removed by the
postinst script. In the title of the progress bar, you should add the
word UrbanTerror.

Cheers,

 Gonéri


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



RFS: urbanterror-data

2008-03-13 Thread Jack Coulter
Dear mentors,

I am looking for a sponsor for my package urbanterror-data.
This package used to be a large (750mb) package including all data
files, it is now simply a downloader package.

* Package name: urbanterror-data
  Version : 4.1
  Upstream Author : FrozenSand LLC [EMAIL PROTECTED]
* URL : http://www.urbanterror.net
* License : Quake 3 SDK EULA/Various free-to-distribute licenses
  Section : non-free/games

It builds these binary packages:
urbanterror-data - Urban Terror Data

The package appears to be lintian clean.

The upload would fix these bugs: 468325

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/non-free/u/urbanterror-data
- Source repository: deb-src http://mentors.debian.net/debian unstable
main contrib non-free
- dget
http://mentors.debian.net/debian/pool/non-free/u/urbanterror-data/urbanterror-data_4.1.dsc

I would be glad if someone uploaded this package for me.

Kind regards
 Jack Coulter


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Re: RFS: urbanterror-data

2008-03-13 Thread Gonéri Le Bouder
On Thu, Mar 13, 2008 at 10:31:31PM +1100, Jack Coulter wrote:
 Dear mentors,
 
Hi,

Just some remarks:

You're archive is not clean:
$tar tf urbanterror-data_4.1.tar.gz| grep tmp
urbanterror-data-4.1/tmp.wSEBJ15633/
urbanterror-data-4.1/tmp.ZDDmV19344/
urbanterror-data-4.1/tmp.OAewO17038/

Lintian also generates some warnings:
W: urbanterror-data: malformed-title-in-templates
urbanterror-data/license
W: urbanterror-data: too-long-extended-description-in-templates
urbanterror-data/license
W: urbanterror-data: missing-debconf-dependency-for-preinst

About the preinst script:
You package should depend on debconf, unzip, wget and zenity since you use
them. You can't be sure the dependency will be correctly installed
without a Pre-Depends fields.
Take a look on Debian Policy 7.2, IMO you should do a postinst instead.

The binary-common section of your rules file is useless. The -a parameter used 
with
debhelpers is also useless since you build just one archindep package.
The get-orig-source: seems to be useless too. You don't need the upstream 
archive
for the build.

Cheers,

 Gonéri


signature.asc
Description: Digital signature


Re: RFS: urbanterror-data

2008-03-13 Thread Gonéri Le Bouder
On Thu, Mar 13, 2008 at 03:27:20PM +0100, Gonéri Le Bouder wrote:
 On Thu, Mar 13, 2008 at 10:31:31PM +1100, Jack Coulter wrote:
  Dear mentors,
  
 Hi,
 
 Just some remarks:
 
 You're archive is not clean:
 $tar tf urbanterror-data_4.1.tar.gz| grep tmp
 urbanterror-data-4.1/tmp.wSEBJ15633/
 urbanterror-data-4.1/tmp.ZDDmV19344/
 urbanterror-data-4.1/tmp.OAewO17038/
 
 Lintian also generates some warnings:
 W: urbanterror-data: malformed-title-in-templates
 urbanterror-data/license
 W: urbanterror-data: too-long-extended-description-in-templates
 urbanterror-data/license
 W: urbanterror-data: missing-debconf-dependency-for-preinst
 
 About the preinst script:
 You package should depend on debconf, unzip, wget and zenity since you use
 them.
zenity needs a X server. I don't use it myself but ssft is probably more
suitable here.

About the package itself, can you call wget with -C flag to be able to
continue a download.

Cheers,

Gonéri


signature.asc
Description: Digital signature


RFS: urbanterror-data

2008-03-03 Thread Jack Coulter
Dear mentors,

I am looking for a sponsor for my package urbanterror-data.

* Package name: urbanterror-data
  Version : 4.1-3
  Upstream Author : FrozenSand LLC [EMAIL PROTECTED]
* URL : http://www.urbanterror.net
* License : Quake 3 SDK EULA/Various free-to-distribute licenses
  Section : non-free/games

It builds these binary packages:
urbanterror-data - Urban Terror Data

The package appears to be lintian clean.

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/non-free/u/urbanterror-data
- Source repository: deb-src http://mentors.debian.net/debian unstable
main contrib non-free
- dget
http://mentors.debian.net/debian/pool/non-free/u/urbanterror-data/urbanterror-data_4.1-3.dsc

I would be glad if someone uploaded this package for me.

Kind regards
 Jack Coulter


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]