Page 6 of 138

Re: Chill's patchpack v6_3

Posted: 23 May 2010 22:06
by NekoMaster
Hmm, well Im deciding on normal + puffs of diesel fumes or always throwing off puffs of diesel smoke. i guess really I should say, choose the one thats easier to do.

Edit : I tried compiling that r19891 patch and i got this

Code: Select all

/home/matthew/openttd/src/command_queue.cpp:8:24: error: copy_paste.h: No such file or directory
make[1]: *** [command_queue.o] Error 1
make[1]: Leaving directory `/home/matthew/openttd/objs/release'
make: *** [all] Error 2
And no binary

Re: Chill's patchpack v6_3

Posted: 23 May 2010 23:33
by bluebottle
This is a build of r19891 for Ubuntu 9.10 Linux (Karmic Koala), using Chillcore's great patch.
  • As with the distro OpenTTD package, you need to have the data files in /usr/share/games/openttd/data/.
  • This build is unstripped to provide better (i.e. useful) core dumps/backtraces. This is similar to including the .pdb file for MSVC builds.
Download: openttd_chillcore-r19891-unstripped-i386.deb

I didn't get nekomaster's copy_paste.h error, but I did get a few warnings I haven't seen before:

Code: Select all

src/copy_paste.cpp: In member function ‘void CopyPaste::internal_PasteArea(TileIndex)’:                                                                                                 
src/copy_paste.cpp:716: warning: name lookup of ‘tile’ changed                                                                                                                          
src/copy_paste.cpp:514: warning:   matches this ‘tile’ under ISO standard rules                                                                                                         
src/copy_paste.cpp:549: warning:   matches this ‘tile’ under old rules

Re: Chill's patchpack v6_3

Posted: 23 May 2010 23:50
by ChillCore
NekoMaster wrote: Edit : I tried compiling that r19891 patch and i got this
Indeed that is one of the three new files missing in the pach ColdIce has posted.
I have send him a PM but I can not edit his post for him.
Try the version attached to the second post. All new files should be in that version. Please let me know if you have problems with that one also.
bluebottle wrote: I didn't get nekomaster's copy_paste.h error, but I did get a few warnings I haven't seen before:
They are known warnings and they have already been reported in the copy-paste thread.
As Bilbo said they were easy to fix, I have not yet had a look at them ...



Note for the windows binary downloaders:
The windows binary posted by ColdIce should be OK as it is compiled after having applied and bumped the version against r19885.
It is just when ColdIce pulled the patch a few new files were not included in the patch. I have confidence he will replace(*) the diff when he reads my PM.

(*)or remove, as the correct one has been reposted by bluebottle.



Edit:
Forgot to say thank you to bluebottle for the binary.

@bluebottle:
What are the "./configure" options I should set for compiling a unstripped binary myself? I tried searching but did not get any results.
I am still on Ubuntu 9.04 and I kind of messed up my installation in /usr/share/games/openttd/data/.
When I tried installing a .deb the last time I got a "is already installed" warning and if I remove OpenTTD and redownload from the software repository the version available is still 6.3 .
Also it gets installed as root and when trying to run OpenTTD from the console I always get warnings about "no available videodrivers" and such.
I am still a newb when it comes to linux and its data structure. :)

Re: Chill's patchpack v6_3

Posted: 24 May 2010 01:20
by NekoMaster
you know you can extract the contents of a deb using ubuntus archive manager, thats what I had to do since openttd is on my flash drive as it has been for over a year now

Re: Chill's patchpack v6_3

Posted: 24 May 2010 07:08
by ColdIce
my bad for the patch :oops: I forgot to do something. I removed the patch file because bluebottle provided a PROPER one.

Re: Chill's patchpack v6_3

Posted: 24 May 2010 08:43
by bluebottle
ChillCore wrote:They are known warnings and they have already been reported in the copy-paste thread.
As Bilbo said they were easy to fix, I have not yet had a look at them ...
OK. It was my first time compiling your patch pack. :)
ChillCore wrote:@bluebottle:
What are the "./configure" options I should set for compiling a unstripped binary myself? I tried searching but did not get any results.
I am still on Ubuntu 9.04 and I kind of messed up my installation in /usr/share/games/openttd/data/.
When I tried installing a .deb the last time I got a "is already installed" warning and if I remove OpenTTD and redownload from the software repository the version available is still 6.3 .
Also it gets installed as root and when trying to run OpenTTD from the console I always get warnings about "no available videodrivers" and such.
I am still a newb when it comes to linux and its data structure. :)
Adding "--disable-strip" to your configure command should do it. It does make a bigger executable though. I build using dpkg-buildpackage and get 36MB executables. 8o

As for your /usr/share/games/openttd/data/, this is my listing:

Code: Select all

copypaste.grf
flat_blacktiles.grf
no_sound.obs
openttdd.grf
openttdw.grf
opntitle.dat
orig_dos_de.obg
orig_dos.obg
orig_dos.obs
orig_win.obg
orig_win.obs
sample.cat
trg1r.grf
trgcr.grf
trghr.grf
trgir.grf
trgtr.grf
Nothing else needs to be there.


In any case, welcome to Linux! You can install any build of OpenTTD over the distro one by using (as root) "dpkg -i <filename>", as long as it was compiled for your distribution & version. You can usually ignore version warnings like "already installed". When you run the game, it must be run as your normal user, not root. A message about "no available video drivers" probably means you didn't have the SDL development packages or the like installed when you compiled, as they are needed to build that driver. I highly recommend using the build instructions on the wiki for building .deb files, as then all the dependencies will be taken care of (or at least, you get told about them). This is my build routine, run in the directory with the trunk OpenTTD source tree:

Code: Select all

patch -p0 < ../***patch name***
mv os/debian .
export DEB_BUILD_OPTIONS=nostrip
dpkg-buildpackage -rfakeroot -uc -us -t
That will give you a nice warm .deb in the parent directory, which you then just install with "dpkg -i". The "export" line is a way to tell the Debian package builder to not strip the executable, so you don't need to bother with any configure scripts. Only the installation is done as root; all compiling is done as your normal user.

Hope that helps, and please keep up the great work with the patch pack. :bow:

Re: Chill's patchpack v6_3

Posted: 24 May 2010 23:47
by ChillCore
NekoMaster wrote: you know you can extract the contents of a deb using ubuntus archive manager, thats what I had to do since openttd is on my flash drive as it has been for over a year now
I did not know that. Cool I will give that a try.
ColdIce wrote: my bad for the patch :oops: I forgot to do something. I removed the patch file because bluebottle provided a PROPER one.
No need for blushing. Everybody makes misstakes. With 23 new files it is easy to miss a few. The error has been corrected that is the most important thing.
bluebottle wrote:
ChillCore wrote: They are known warnings and they have already been reported in the copy-paste thread.
As Bilbo said they were easy to fix, I have not yet had a look at them ...
OK. It was my first time compiling your patch pack. :)
Better to have the same thing reported a few times than not having bugreports at all. :)
bluebottle wrote: Many useful comments regarding unstripped binaries and .deb
I had to install dpkg and I have removed the OpenTTD version that I got from the linux software repository.
I can now run OpenTTD from the console.

I do however still have to figure some stuff out in the config file to be able to make a debian package as it exits with a few warnings. Jippie.


ps:
Sorry for the late reply.
My wireless internet connection ... I want cable when my subscription is expired.

Re: Chill's patchpack v6_3

Posted: 05 Jun 2010 14:41
by ReisRyos
I like this path :D

Re: Chill's patchpack v6_3

Posted: 05 Jun 2010 19:29
by NekoMaster
What path? theres no path here, only Patchs :3

Re: Chill's patchpack v6_3

Posted: 06 Jun 2010 16:55
by petert
NekoMaster wrote:What path? theres no path here, only Patchs :3
That's a bit unneeded, imo. Clearly he meant 'patch'.

Re: Chill's patchpack v6_3

Posted: 06 Jun 2010 18:17
by NekoMaster
petert wrote:
NekoMaster wrote:What path? theres no path here, only Patchs :3
That's a bit unneeded, imo. Clearly he meant 'patch'.
I was only joaking

Re: Chill's patchpack v6_3

Posted: 11 Jun 2010 05:46
by ReisRyos
Chillcore: When will you are planning another update :D ?
Thanks

Re: Chill's patchpack v6_3

Posted: 11 Jun 2010 13:33
by ChillCore
Version 7 will be coming soon. I did not have much time to code lately as I have been playing a bit myself. :)
I am updating the More height levels patch a bit and I have already updated cargodist to r19940.

It will come with a little scenario that I am first playtesting and adjusting before releasing it.
Also it will break savegame compatibility with previous versions due to some changes in Cargodist.

Thank you for your kind words a few posts back.

Re: Chill's patchpack v6_3

Posted: 11 Jun 2010 16:14
by NekoMaster
Because Im back on windows I hope that someone can make a win32 build when that update comes.

Re: Chill's patchpack v6_3

Posted: 25 Jun 2010 08:41
by ReisRyos
Update? Please?

Re: Chill's patchpack v6_3

Posted: 27 Jun 2010 18:38
by ChillCore
So finally, here is v7.

Changes from v6_3:
Cargodist updated to r20016.
More height levels updated to v27_4_r20026.

Enjoy.
First posts will be updated in a bit.


ps:
Please do not make asking for updates a habbit.
I know you are eager to play with the new features and bugfixes available in trunk, so am I, but updating the patchpack is a bit more work then just hitting the magic button.
Also my first priority is the More height levels patch.(First finishing it then getting it in trunk bit by bit, hopefully)
Thank you.

Re: Chill's patchpack v6_3

Posted: 27 Jun 2010 20:06
by Grandmaster
is it possible to include this patch. Close adjacent level crossings.
http://www.tt-forums.net/viewtopic.php? ... 91&start=0

Re: Chill's patchpack v6_3

Posted: 27 Jun 2010 20:37
by ChillCore
I had a quick looksie at the code that patch changes.
It should not be a problem to add it as the code to change is not yet altered by another patch in the patchpak if my memory serves me well.

I will first put it in my test folder to test, as I have not yet played with that patch yet.
Also Eddi mentioned that he could break it with some effort ... if it is not to easy to do so it will be in the next version.

Thank you for the suggestion, it seems to be a nice feature and I somehow missed it when it was posted.

Re: Chill's patchpack v6_3

Posted: 27 Jun 2010 21:36
by NekoMaster
So, I was wondering if someone when they have time can compile a win32 build for people that can't compile on windows (I know theres quite a few people out there that are like that, as for me, I can't afford to download the stuff to compile win32 builds)

Re: Chill's patchpack v6_3

Posted: 27 Jun 2010 21:55
by petert
NekoMaster wrote:I can't afford to download the stuff to compile win32 builds
What "stuff" is that? The MSYS guide doesn't require many things - if it still works.