[debian] howto: rebuilding sgt-puzzles for the FR

[EdorFaus]

On 12/17/2010 12:08 AM, Neil Jerram wrote:
> For people who like Simon Tatham's puzzle collection...
>
> On my SHR-T, for a long time I've used Frode Austvik's sgt-puzzles

Thank you! You just made me go back to look at this code again.

When I first started doing this package, I fully intended to keep 
maintaining it - but then several things conspired against it, which has 
caused it to be unmaintained (by me at least) for a year now...
I do hope to get back into things with my FR, but... well, we'll see.


> evening I finally realised that (a) the necessary code, i.e. Frode
> Austvik's patches, is all already in the Debian source package; and

Actually, most of them have been included upstream, which is probably 
where Debian got them from... I just removed a bunch of patches from my 
repository because they're no longer needed. :)

I've still got a couple left though... and made a new one just now... I 
should probably try to push those upstream as well.

Looking at the Debian source package, it has several other patches I 
don't, that could be quite useful on the FR (I'm not sure what's 
actually used and not, but one patch mentions using a shared lib for the 
shared code, and another mentions building all games into one binary to 
share the code that way... either could be nice for space).

I should probably have based mine on the deb package to start with, 
instead of reinventing stuff... ohwell, at least this way I built trunk, 
and had an idea of what patches are actually used.


> (b) it's quite easy to rebuild the package (on the FR) to enable it:

Heh, this is something I hadn't really considered - building it *on* the 
FR. :) Thinking about it though, that would probably make several issues 
I had with building it easier - such as generating the icons (which 
requires an X server, and hence binaries native to the compiling 
system... which caused me to compile twice when cross-compiling :P ).


> # vi puzzles.h
>
> Comment out the "#ifdef _WIN32_WCE" and "#endif" around a group of
> defines for "Pocket PC devices", which are equally applicable to the
> FR.

On 12/17/2010 12:59 PM, Timo Juhani Lindfors wrote:
 > Could you make this a command line option?

It already is, sortof - basically, replace the "debian/rules binary" 
command with this one:

CFLAGS="-D_WIN32_WCE" debian/rules binary

-- 
Regards,
Frode