Package: sgt-puzzles / 20191231.79a5378-3

Metadata

Package Version Patches format
sgt-puzzles 20191231.79a5378-3 3.0 (quilt)

Patch series

view the series file
Patch File delta Description
0001 Apply version string substitutions from the tarball.patch | (download)

configure.ac | 2 1 + 1 - 0 !
devel.but | 2 2 + 0 - 0 !
osx-info.plist | 4 2 + 2 - 0 !
puzzles.but | 2 2 + 0 - 0 !
version.h | 13 2 + 11 - 0 !
5 files changed, 9 insertions(+), 14 deletions(-)

 apply version string substitutions from the tarball


102_fix pearl min dimensions.diff | (download)

pearl.c | 2 2 + 0 - 0 !
1 file changed, 2 insertions(+)

 pearl: require width or height to be at least 6 for tricky

Bug-Debian: https://bugs.debian.org/667963

Josh Triplett reported:
> If I ask pearl to generate a 5x5 tricky puzzle, it runs forever.

I find that 5x6 or 6x5 works, so set the minimum accordingly.

fix ftbfs with gcc 6.patch | (download)

towers.c | 12 6 + 6 - 0 !
1 file changed, 6 insertions(+), 6 deletions(-)

 fix ftbfs with gcc 6

Bug-Debian: https://bugs.debian.org/811577

gcc 6 warns about statements that are indented as if they were meant to be
part of a preceding conditional block.  In this case I don't think that was
intended, so shift it left.

0004 Makefile.doc Remove CHM files in clean rule.patch | (download)

Makefile.doc | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 makefile.doc: remove chm files in clean rule


206_translate docs.diff | (download)

.gitignore | 3 3 + 0 - 0 !
Makefile.doc | 31 31 + 0 - 0 !
po/puzzles-doc.pot | 6339 6339 + 0 - 0 !
po/puzzles-doc.pot.head | 7 7 + 0 - 0 !
4 files changed, 6380 insertions(+)

 support translated docs using po4a

Bug-Debian: https://bugs.debian.org/483665

0006 Add German translation of documentation.patch | (download)

po/de.po | 8247 8247 + 0 - 0 !
1 file changed, 8247 insertions(+)

 add german translation of documentation

This was done by Helge Kreutzmann.

201_make more docs.diff | (download)

.gitignore | 4 3 + 1 - 0 !
Makefile.doc | 36 36 + 0 - 0 !
mkmanpages.pl | 157 157 + 0 - 0 !
3 files changed, 196 insertions(+), 1 deletion(-)

 add rules and script to build manual pages and html

Halibut already supports these formats but since the documentation is
all combined we need to do a bit more work to extract the right
information for each game's manual page.

202_online help.diff | (download)

Recipe | 1 1 + 0 - 0 !
gtk.c | 109 109 + 0 - 0 !
2 files changed, 110 insertions(+)

 add html-based online help

This works along the same lines as the Windows implementation,
though we have to try a bit harder to find a help browser.

207_slant shade filled.diff | (download)

slant.c | 6 1 + 5 - 0 !
1 file changed, 1 insertion(+), 5 deletions(-)

 slant: shade filled squares

Bug-Debian: https://bugs.debian.org/419836

303_show debian version number.diff | (download)

version.c | 6 5 + 1 - 0 !
1 file changed, 5 insertions(+), 1 deletion(-)

 show debian package version number

Include Debian version number in any version display to make
it obvious that the binaries are built from modified source.

0011 mkfiles.pl Change first line to usr bin perl.patch | (download)

mkfiles.pl | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 mkfiles.pl: change first line to #!/usr/bin/perl
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Debian policy §10.4 says it must be this and not "#!/usr/bin/env perl".

[git-debrebase: split mixed commit: upstream part]

0012 unruly Fix memory leak in dup_game.patch | (download)

unruly.c | 19 11 + 8 - 0 !
1 file changed, 11 insertions(+), 8 deletions(-)

 unruly: fix memory leak in dup_game()

The common structure is ref-counted and dup_game() bumps the reference
count rather than copying it.  However, blank_state() always allocates
a new instance.  Add a parameter to control whether blank_state()
allocates it.

Fixes: 47cec547e59a ("Unruly, Group: reference-count the 'immutable' array.")

0013 bridges Fix off by one in WITHIN.patch | (download)

bridges.c | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 bridges: fix off-by-one in within()

WITHIN() used to treat the min and max as inclusive bounds but was
changed to treat the max as exclusive, apparently accidentally.

Fixed: 5f5b284c0bdd ("Use C99 bool within source modules.")

0014 pattern Fix build failure with fortify and gcc 9 on .patch | (download)

pattern.c | 3 3 + 0 - 0 !
1 file changed, 3 insertions(+)

 pattern: fix build failure with fortify and gcc 9 on some
 architectures
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Building with gcc 9 and _FORTIFY_SOURCE=2 (the default on Debian)
fails for some architectures, with error messages such as:

In file included from /usr/include/string.h:495,
                 from ./pattern.c:7:
In function ‘memset’,
    inlined from ‘do_row’ at ./pattern.c:456:9:
/usr/include/mips64el-linux-gnuabi64/bits/string_fortified.h:71:10: error: ‘__builtin_memset’ specified size between 18446744071562067968 and 18446744073709551615 exceeds maximum object size 9223372036854775807 [-Werror=stringop-overflow=]
   71 |   return __builtin___memset_chk (__dest, __ch, __len, __bos0 (__dest));
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

The length argument is len, implicitly converted from int to size_t.
Apparently the compiler thinks len might be negative and convert to an
enormous positive value.  Assert that it is not.

0015 Update German translation thanks to Helge Kreutzmann.patch | (download)

po/de.po | 2049 1061 + 988 - 0 !
1 file changed, 1061 insertions(+), 988 deletions(-)

 update german translation, thanks to helge kreutzmann

[git-debrebase split: mixed commit: upstream part]