investigating

Status: NEW => ASSIGNED
Assignee: bugsquad => doktor5000

Hmmm, when not letting it download the .deb packages with the manpages it reports as missing, it doesn't generate any manpage at all. Hence I've created a tarball from the downloaded deb's:

[doktor5000@Mageia4 SPECS]$ tar -tvf ../SOURCES/man-pages-de-1.7-debian-downloads.tar.bz2
drwxrwxr-x doktor5000/doktor5000 0 2014-08-24 22:56 english/downloads/
-rw-rw-r-- doktor5000/doktor5000 962042 2014-08-21 23:32 english/downloads/manpages_3.71-1_all.deb
-rw-rw-r-- doktor5000/doktor5000 323566 2014-08-14 17:57 english/downloads/grep_2.20-2_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 161388 2014-06-02 11:26 english/downloads/mount_2.20.1-5.8_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 234932 2014-07-13 14:51 english/downloads/procps_3.3.9-7_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 374382 2014-05-01 17:21 english/downloads/isdnlog_3.25+dfsg1-3.4_amd64.deb
-rw-rw-r-- doktor5000/doktor5000  13914 2013-11-03 15:49 english/downloads/hostname_3.15_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 341624 2014-07-22 02:16 english/downloads/wodim_1.1.11-3_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 326232 2014-04-21 23:52 english/downloads/recutils_1.7-1_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 130042 2014-08-03 21:06 english/downloads/sysvinit-core_2.88dsf-53.3_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 183636 2014-07-28 12:46 english/downloads/vorbis-tools_1.4.0-1.1_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 193484 2014-08-11 15:59 english/downloads/parted_3.2-4_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 500660 2014-06-02 11:26 english/downloads/util-linux_2.20.1-5.8_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 242372 2014-02-09 18:49 english/downloads/sed_4.2.2-4_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 110640 2013-12-26 00:17 english/downloads/gzip_1.6-3_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 210800 2013-05-06 16:56 english/downloads/bsdmainutils_9.0.5_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 133274 2013-06-06 02:04 english/downloads/less_458-2_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 672754 2014-03-22 23:18 english/downloads/tar_1.27.1-2_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 1860718 2014-08-21 23:32 english/downloads/manpages-dev_3.71-1_all.deb
-rw-rw-r-- doktor5000/doktor5000  205588 2014-08-13 02:59 english/downloads/nfs-common_1.2.8-9_amd64.deb
-rw-rw-r-- doktor5000/doktor5000  265288 2013-09-07 15:02 english/downloads/diffutils_3.3-1_amd64.deb
-rw-rw-r-- doktor5000/doktor5000  112770 2014-07-21 03:57 english/downloads/binkd_1.0.2-1_amd64.deb
-rw-rw-r-- doktor5000/doktor5000  325990 2014-05-09 18:15 english/downloads/fdutils_5.5-20060227-7_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 1088184 2014-06-30 06:18 english/downloads/groff-base_1.22.2-6_amd64.deb
-rw-rw-r-- doktor5000/doktor5000  268836 2014-06-03 08:34 english/downloads/info_5.2.0.dfsg.1-4_amd64.deb
-rw-rw-r-- doktor5000/doktor5000 2583710 2014-04-13 01:44 english/downloads/coreutils_8.21-1.2_amd64.deb
-rw-rw-r-- doktor5000/doktor5000  302948 2013-07-02 23:46 english/downloads/lilo_24.0-2_amd64.deb
-rw-rw-r-- doktor5000/doktor5000  483606 2014-07-06 08:43 english/downloads/findutils_4.4.2-9_amd64.deb

I've also added a basic check if any generated man pages are present at all, that should prevent this issue from reappearing. Thanks for the report.

Now there are 533 man-pages, some are dropped due to the 80% threshold.

Fixed for cauldron with
http://svnweb.mageia.org/packages?view=revision&revision=667236

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

Downloading debs is not needed at all. For Fedora, it is impossible anyway to download any external stuff when building a package on the build servers.

Have a look at the mentioned Fedora package: The %build and %install sections show the correct way to use the system's man pages instead of the Debian ones, I assume you want to have translated versions of those man pages which Mageia ships...? When using the Debian versions, you won't really get what you want, it could happen that Debian has older or newer package versions than Mageia. Just run the usual Autotools tool chain, that's all.

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

Well, it's not downloaded on buildsystem but I've added a static tarball. And when only running make, it does not generate any man-page, only spewing out:

The original manpage for [...] could not be found.

for all original man-pages it's looking for.

Seems po/generate-manpage.sh only checks for raw or .gz versions of original man-pages, hence the checks fail on Mageia, as we use .xz by default since quite some time.

Thanks for the hint, added support for .xz man-pages to po/generate-manpage.sh and dropped that tarball with Debian man-pages. Also noticed that my check for generated man-pages can be simplified, just checking the size of the generated whatis index should be enough.

Please check man-pages-de-1.7-3.mga5 once it hits mirrors.

You may want to check http://svnweb.mageia.org/packages?view=revision&revision=667245 and especially http://svnweb.mageia.org/packages/cauldron/man-pages-de/current/SOURCES/man-pages-de-1.7-mga-check-for-xz-manpages.patch?view=markup&pathrev=667245
The latter was a quick&dirty fix, should probably be extended by bzip2 and .lzma support. Not sure what Fedora or Debian currently uses as default compression ...

How does the saying go: Nothing is as easy as it looks :)

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

(In reply to Florian Hubold from comment #4)
> You may want to check
> http://svnweb.mageia.org/packages?view=revision&revision=667245 and
> especially
> http://svnweb.mageia.org/packages/cauldron/man-pages-de/current/SOURCES/man-
> pages-de-1.7-mga-check-for-xz-manpages.patch?view=markup&pathrev=667245
> The latter was a quick&dirty fix, should probably be extended by bzip2 and
> .lzma support. Not sure what Fedora or Debian currently uses as default
> compression ...
> 
Debian and Fedora still use gz, but at least Fedora is future-proof by using a wildcard for the compression format in the rpm spec files.

BTW, your patch has been committed upstream:
https://alioth.debian.org/plugins/scmgit/cgi-bin/gitweb.cgi?p=manpages-de/manpages-de.git;a=commit;h=79c39f7ecd846ff3c3e7450891adf012ed1d6177

(In reply to Mario Blättermann from comment #5)
> BTW, your patch has been committed upstream:
> https://alioth.debian.org/plugins/scmgit/cgi-bin/gitweb.cgi?p=manpages-de/
> manpages-de.git;a=commit;h=79c39f7ecd846ff3c3e7450891adf012ed1d6177

Thanks a lot :)

I'm still missing some translated man pages from the package. This is due to still insufficient BuildRequires. The package "coreutils" is missing. While figuring out the BR for Fedora and adjust them for Mageia, I forgot to add coreutils, because in Fedora it is a requirement of isdn4k-utils, and in Mageia it isn't.

Please add coreutils to BR, but wait before building a new package, I'm working on some other enhancements which could be also adapted to the Mageia package. BTW, a new upstream version will be published next weeks, in any case before Debian Jessie gets frozen (5 November).

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

Hmmm, should be there, as that's required by basesystem-minimal:

ââ[doktor5000@Mageia4]â[16:09:33]â[~/MGA/CAULDRON/man-pages-de]
ââââ¼ urpmq --requires basesystem-minimal | grep coreutils
--requires behaviour changed, use --requires-recursive to get the old behaviour
coreutils

What binaries from coreutils are really required? Or does it only require the original man-pages? Then coreutils-doc would be needed, as those have been split out into a separate subpackage ...

I've just added this to cauldron¹, as we're currently in version freeze for mga5.
If there's any new release for man-pages-de then feel free to reopen this bug, I'll take care then.

1: http://svnweb.mageia.org/packages?view=revision&revision=736983

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

(In reply to Florian Hubold from comment #8)
> What binaries from coreutils are really required? Or does it only require
> the original man-pages? Then coreutils-doc would be needed, as those have
> been split out into a separate subpackage ...
> 
man-pages-de doesn't need any binaries, only the man pages are required. So in general, if any of the required packages has man pages in a subpackage, we need that one, even without the base package.

> I've just added this to cauldron¹, as we're currently in version freeze for
> mga5.
> If there's any new release for man-pages-de then feel free to reopen this
> bug, I'll take care then.
> 
OK, the additional BR doesn't break a software freeze, because we don't change the version number. It's just an enhancement and could be pushed to mga5, as far as I can evaluate.

I've added some more things to the latest Fedora package for v1.7 [1]:

* The file cal.1.po was broken, I've backported a patch from Git.
* The gettext string with version info remains untranslated in almost all cases due to differing version numbers in the tarball and the package man-pages on the target system. A sed command and some %global definitions fix this.

For future releases: Once procps-ng-3.3.10 has arrived in the Mageia repos, it will cause file conflicts with man-pages-de. Then the files po/man1/{free.1.po,uptime.1.po} have to be removed, also procps-ng from BR. The new procps version ships its own translated man pages, so those from man-pages-de will become obsolete. See also [2].

[1] http://pkgs.fedoraproject.org/cgit/man-pages-de.git/commit/?h=f20&id=c812992471e9dde65d647f22a8989985ac927b53
[2] https://bugzilla.redhat.com/show_bug.cgi?id=1149306

(In reply to Mario Blättermann from comment #9)
> (In reply to Florian Hubold from comment #8)
> > I've just added this to cauldron¹, as we're currently in version freeze for
> > mga5.
> > If there's any new release for man-pages-de then feel free to reopen this
> > bug, I'll take care then.
> > 
> OK, the additional BR doesn't break a software freeze, because we don't
> change the version number. It's just an enhancement and could be pushed to
> mga5, as far as I can evaluate.

Sorry for the confusion. I've already submitted man-pages-de with BuildRequires on coreutils-doc. I only meant for the upcoming version update you wrote about, for this I'll need to submit a freeze push request.

All good - simply reopen this bug when you push the next man-pages-de version :)

Just FYI: There were a few new versions released since 1.11, see http://manpages-de.alioth.debian.org/downloads/

Status: RESOLVED => REOPENED
CC: (none) => mario.blaettermann
Resolution: FIXED => (none)

Thanks for letting us know, though unless the current package lacks translated packages again, it's not a valid reason for reopening a fixed bug :)

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

(In reply to Florian Hubold from comment #10
> 
> All good - simply reopen this bug when you push the next man-pages-de
> version :)

@ Akien

in this case it looks like a valid reason ;)

Status: RESOLVED => REOPENED
CC: (none) => marja11
Resolution: FIXED => (none)

My bad :)

(In reply to Mario Blättermann from comment #11)
> Just FYI: There were a few new versions released since 1.11, see
> http://manpages-de.alioth.debian.org/downloads/

Hi Mario, thanks for the ping, will take a look :)

Status: REOPENED => ASSIGNED

Any news about that? I still see v1.11 in Cauldron. Some new versions have been released in the meantime, the latest is 1.22.

1.22 is in cauldron since a while thanks to Thierry. Sorry for the late reply :/

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