Bug#775899: minetest: status of forwarded patches

[Markus Koschany]

Source: minetest
Version: 0.4.11+repack-1
Severity: normal

Hi,

this is a bug report to track the status of all minetest patches
currently applied by Debian. It would be a good idea to drop,
simplify or forward some of them.

This bug report should be ideally closed with the next upstream release.

Here is the list of Debian patches:

debianization.patch:
    Forwarded: https://github.com/minetest/minetest/issues/2163

desktop-file-fix-icon.patch:
    Should be dropped. The icon name should not be renamed. Instead we
    need to update debian/rules.

desktop-file.patch:
    Forwarded: https://github.com/minetest/minetest/pull/2165
    Already accepted. Can be dropped with the next upstream release.

dynamic-link.patch:
    Not forwarded yet. Same as debianization.patch. Should be fixed upstream.

gcc47.patch:
    Is this patch still relevant?

hurd.patch:
    Forwarded: https://github.com/minetest/minetest/pull/2162

hyphen-used-as-minus.patch:
    Forwarded: https://github.com/minetest/minetest/pull/2166
    Already accepted.

i18n.patch:
    Not forwarded yet. The patch looks wrong somehow. Why can't we
    just use the build system for adjusting the path instead?

kfreebsd-gettext.patch:
    Not forwarded yet. This patch should be updated and then forwarded.
    It has to be more generic because otherwise it would conflict with
    other BSD systems like OpenBSD, FreeBSD, etc. There must be a way
    to use a conditional only for kfreeBSD.

mapper.patch:
    This one uses a hardcoded path. I think we should file an upstream
    bug report because it should be possible to adjust the path to
    colors.txt when compiling minetest. In its current form the patch
    is not upstreamable.

rawlua.patch:
    Not forwarded yet. The patch should be more generic. Same as
    debianization.patch and dynamic-link.patch. That's a problem with
    upstream's build system and should be fixed but the current patch
    won't probably be accepted.

shared_mods.patch:
    Not forwarded. Another hardcoded path to the systemwide mod directory. This looks
    wrong to me. Shouldn't it be possible to adjust the path at build
    time?

useless-depends:
    We should consider to drop this patch because now we use
    export DEB_LDFLAGS_MAINT_APPEND = -Wl,--as-needed in debian/rules
    which might achieve the same.


Markus