Bug 13414 - new version urbanterror (but also preferably without autodownloader)
Summary: new version urbanterror (but also preferably without autodownloader)
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Juan Luis Baptiste
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-05-20 09:13 CEST by AL13N
Modified: 2015-05-19 19:08 CEST (History)
2 users (show)

See Also:
Source RPM: urbanterror
CVE:
Status comment:


Attachments

Description AL13N 2014-05-20 09:13:34 CEST
Description of problem:
urbanterror install user-based (instead of system-based) and to top it off, updates will get rid of settings.

i propose to use the opensuse schema and to renew urbanterror, there is an urbanterror-data package there too, which is Freeware iinm, but should be checked.



Reproducible: 

Steps to Reproduce:
Rémi Verschelde 2014-05-20 20:11:39 CEST

CC: (none) => remi
Assignee: bugsquad => juan.baptiste

Comment 1 Juan Luis Baptiste 2014-05-20 21:49:07 CEST
This is already on the works, I have the packages ready, the main package is already imported as urbanterror42 (but never pushed to the BS), they just need to be updated to the latest version. The problem now is that after a hard drive crash I can't find the data package spec on my backups, still looking !!

Status: NEW => ASSIGNED

Comment 2 AL13N 2014-05-21 07:57:06 CEST
is it needed to push as urbanterror42 instead of urbanterror? people having the urbanterror game probably want to have the new version...
Comment 3 Juan Luis Baptiste 2014-05-21 15:28:07 CEST
No, the reason why I named the packages differently was because I started to work on them more than a year ago, when there still were both 4.1.1 and 4.2 (as a beta) versions available, and at the time it wasn't clear when 4.2 was going to become final and 4.1.1 servers were going to be shut down. This happened until last December.
Comment 4 AL13N 2014-05-21 15:47:47 CEST
if you don't have the -data .spec file anymore, you could take a peek at opensuse, i looked at them, cause i was planning to do it if there was no maintainer and it looks like doing like opensuse would mean less maintenance...
Comment 5 AL13N 2014-05-21 15:48:22 CEST
also, i should note that it would be best to commit this to urbanterror on cauldron only...
Comment 6 Juan Luis Baptiste 2014-05-21 23:13:50 CEST
The updated packages are ready, I need to test them first and then replace remove urbanterror42 from svn, update urbanterror and import urbanterror-data. 

I think we should also push an update for mga 3, currently the version from that release is unusable because the game master server only answers to clients with version >= 4.2.x.
Comment 7 AL13N 2014-05-22 09:10:23 CEST
i don't agree, but that's another discussion for another bug report:
- this is because if you are doing lan plays, and you have a 4.1 server it still works well

possibly a backport would be doable.
Comment 8 AL13N 2014-05-22 10:13:56 CEST
btw: i have 2 people who are waiting on 4.2 :-) so they can test at least... :-)
Comment 9 AL13N 2014-05-22 13:18:49 CEST
i stand corrected, there is some kind of bug which even disallows local LAN server-client...

we tested at work during lunch-break and server running 4.1 version, all clients get (server not listed due to higher than 800ms).

so, we do need update on mga3 and mga4
Comment 10 AL13N 2014-05-26 12:21:00 CEST
if you could have a build submitted to updates_testing, we can test it for you...
Comment 11 Juan Luis Baptiste 2014-05-26 17:36:51 CEST
Last night I got this error while importing the -data package:

[juancho@dci-laptop rpmbuild]$ mgarepo import /home/juancho/rpmbuild/SRPMS/urbanterror-data-4.2.018-1.mga4.src.rpm
Importing package to svn+ssh://svn.mageia.org/svn/packages/cauldron/urbanterror-data
error: command failed: cp -rf /tmp/tmpMtm1Zz/SOURCES /tmp/tmpG8t4wd/current
cp: error writing '/tmp/tmpG8t4wd/current/SOURCES/UrbanTerror42_full018.zip': No space left on device
cp: failed to extend '/tmp/tmpG8t4wd/current/SOURCES/UrbanTerror42_full018.zip': No space left on device


It seems that the svn server doesn't have enough free space in /tmp ?
Comment 12 AL13N 2014-05-27 09:35:21 CEST
that sure looks like it... perhaps you can contact sysadm@ ML?
Comment 13 Colin Guthrie 2014-05-27 10:08:17 CEST
FWIW, /tmp has about 94Gb free.

I'm told the urganterror-data package is about 1.5Gb (ouch for upload!), so we probably have at least three copies of this data... one in the srpm file itself, one in the extracted version (/tmp/tmpMtm1Zz/SOURCES) and one when copying things to the svn structure (/tmp/tmpG8t4wd/current). But even then, using up 94Gb is a pretty tall order!!

I should think that at least the second command could use hardlinks (cp -rlf) to save space (and time) but even then I'm surprised that the problem balloons the disk space so much....

Not quite sure what to suggest for now.

CC: (none) => mageia

Comment 14 Colin Guthrie 2014-05-27 10:20:34 CEST
D'oh!

The /tmp it's referring to is *your* /tmp!

As we use tmpfs these days, the fact it's run out of space is not surprising (the cp -alr thing would help here but still might not have enough room).

Quick fix is to just run the command as:

TMPDIR=/some/real/path/with/plenty/space mgarepo putsrpm /home/juancho/rpmbuild/SRPMS/urbanterror-data-4.2.018-1.mga4.src.rpm


That should do it!

[Aside: Strangely enough I thought TMPDIR used to be set to $HOME/tmp... but I seem to have it set to /tmp here too... a puzzle for another day]
Comment 15 Juan Luis Baptiste 2014-05-27 23:13:17 CEST
Colin thanks for the tip !! I didn't realize that as I had around 3GB of free space on my /tmp, so I thought the error was on the server.

I have now uploaded urbanterror-data and updated urbanterror, but they're still not ready to be pushed to cauldron, I'm missing a README.urpmi file with the notes about the old package that autowdownloaded the game data into .q3ut4.
Comment 16 AL13N 2014-05-30 10:30:52 CEST
i see that you've committed, i'm gonna build a local build and test it.

i can probably add the README.urpmi if you like... it's just gonna take some time before the urbanterror-data is checked out...
Comment 17 Rémi Verschelde 2014-05-30 10:42:54 CEST
You chould check out with "mgarepo co -s urbanterror-data" to download only the spec file, and then commit and submit your changes.
Comment 18 AL13N 2014-05-30 11:26:44 CEST
i did one better and had the binary already and put that in place...

i also did the commit to add a README.urpmi , go check it out and correct where necessary.

i also note you had a %changelog section in urban-terror, because it's still the same as the opensuse one. please clean that up. (there is a way to keep history, if you want to, check the wiki)

in any case, i'm building rpms, so i can test locally...

i'll let you know if i got it working, (i'll test on mga3 and mga4, don't have cauldron lying about).
Comment 19 AL13N 2014-05-30 14:25:29 CEST
single player (local server + some bots) seem to work fine, on mga3... my collegue on mga4 will install this later and then we can see if we can connect to each other...
Comment 20 AL13N 2014-05-30 14:27:57 CEST
i notice that urbanterror-data package depends on the urbanterror package ? shouldn't this be suggested?
Comment 21 AL13N 2014-05-30 15:57:04 CEST
we had some trouble connecting to each other, specifically, we always see that 1 local server is filtered in the logs... maybe i did a bad configuration somewhere.

direct connect to local server worked, so i'll guess that's good enough for me...
Comment 22 Juan Luis Baptiste 2014-05-31 00:51:55 CEST
(In reply to AL13N from comment #20)
> i notice that urbanterror-data package depends on the urbanterror package ?
> shouldn't this be suggested?

No because urbanterror-data is useless without the game binaries from urbanterror package, the same the other way around.
Comment 23 Juan Luis Baptiste 2014-05-31 06:01:36 CEST
@AL13N: are you sure you commited the README.urpmi file ? I just checked out urbanterror-data and on SOURCES there's only the game zip file.
Comment 24 AL13N 2014-05-31 10:58:20 CEST
README.urpmi are often generated through the spec file....
Comment 25 AL13N 2014-05-31 11:00:22 CEST
(In reply to Juan Luis Baptiste from comment #22)
> (In reply to AL13N from comment #20)
> > i notice that urbanterror-data package depends on the urbanterror package ?
> > shouldn't this be suggested?
> 
> No because urbanterror-data is useless without the game binaries from
> urbanterror package, the same the other way around.

i know this is nitpicking, but theoretically, one could use the data package for editing, and i'm unsure whether cyclic dependencies is what's wanted...
Comment 26 Juan Luis Baptiste 2014-06-03 23:12:27 CEST
(In reply to AL13N from comment #24)
> README.urpmi are often generated through the spec file....

Sorry, I didn't saw it in the spec the first time I looked. The content is fine.



(In reply to AL13N from comment #25)
> (In reply to Juan Luis Baptiste from comment #22)
> > (In reply to AL13N from comment #20)
> > > i notice that urbanterror-data package depends on the urbanterror package ?
> > > shouldn't this be suggested?
> > 
> > No because urbanterror-data is useless without the game binaries from
> > urbanterror package, the same the other way around.
> 
> i know this is nitpicking, but theoretically, one could use the data package
> for editing, and i'm unsure whether cyclic dependencies is what's wanted...

If you do map editing, you probably would need the game binaries to check your changes. Besides, Urban Terror doesn't include any edition tools.
Comment 27 AL13N 2014-06-04 09:51:49 CEST
anyway, it worked on my build, so you can submit it just fine... (and request backport too)
Comment 28 Juan Luis Baptiste 2014-06-04 18:37:55 CEST
Pushed to nonfree, and for mga 4 on this case this would go as an update, not a backport.
Comment 29 Samuel Verschelde 2015-05-19 19:08:21 CEST
Fixed in cauldron according to comment #28. Please open another bug report if an update to Mageia 4 is needed.

Status: ASSIGNED => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.