Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 482748

Summary: app-text/a2ps-4.14-r3 with >=sys-apps/texinfo-5.2 - a2ps.texi:422: @uref missing close brace (possibly involving @href) -/ /- make[2]: *** [a2ps.info] Error 1
Product: Gentoo Linux Reporter: Guy <defuebr>
Component: [OLD] PrintingAssignee: Printing Team <printing>
Status: RESOLVED FIXED    
Severity: normal CC: alexandref75, bircoph, bkohler, cjk
Priority: Normal Keywords: PATCH
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 464362    
Attachments: /var/tmp/portage/app-text/a2ps-4.14-r3/temp/build.log
emerge --info '=app-text/a2ps-4.14-r3::gentoo'
corrects compilation with texinfo 5.x
ebuild that applies the textinfo patch
a2ps-4.14-r4.ebuild.diff
a2ps-4.14-r4.ebuild.diff

Description Guy 2013-08-28 01:44:42 UTC
Created attachment 357218 [details]
/var/tmp/portage/app-text/a2ps-4.14-r3/temp/build.log

This is may be a regression related to fixed bug #335803.

However, no error message appears when patch 'a2ps-4.14-ja-cleanup.patch' is applied. The compile seems to fail with apparent syntax issues when 'a2ps.texi' is processed.


>>> Preparing source in /var/tmp/portage/app-text/a2ps-4.14-r3/work/a2ps-4.14 ...
 * Applying a2ps-4.13c-locale-gentoo.diff ...
 [ ok ]
 * Applying a2ps-4.13-stdout.diff ...
 [ ok ]
 * Applying a2ps-4.14-ja_nls.patch.gz ...
 [ ok ]
 * Applying a2ps-4.14-ja-cleanup.patch ...
 [ ok ]
 * Applying a2ps-4.13c-fnmatch-replacement.patch ...
 [ ok ]
 * Applying a2ps-4.14-psset.patch ...
 [ ok ]
 * Applying a2ps-4.13c-emacs.patch ...
 [ ok ]
 * Applying a2ps-4.13-manpage-chmod.patch ...
 [ ok ]
 * Applying a2ps-4.14-check-mempcpy.patch ...
 [ ok ]
 * Applying a2ps-4.14-fix-stpcpy-proto.patch ...
 [ ok ]
 * Applying a2ps-4.14-ptrdiff_t.patch ...
 [ ok ]

. . .

a2ps.texi:47: warning: redefining Texinfo language command: @LaTeX
a2ps.texi:422: @uref missing close brace (possibly involving @href)
a2ps.texi:420: @uref missing close brace (possibly involving @href)
a2ps.texi:435: misplaced }
a2ps.texi:521: @uref missing close brace (possibly involving @href)
a2ps.texi:2800: misplaced }
a2ps.texi:4122: misplaced }
a2ps.texi:5669: @end example seen before @var closing brace
a2ps.texi:5677: @var missing close brace
a2ps.texi:5871: @end example seen before @var closing brace
a2ps.texi:5876: @var missing close brace
a2ps.texi:6219: @end quotation seen before @i closing brace
a2ps.texi:6348: @end quotation seen before @i closing brace
a2ps.texi:6588: @end quotation seen before @i closing brace
Comment 1 Guy 2013-08-28 01:47:42 UTC
Created attachment 357220 [details]
emerge --info '=app-text/a2ps-4.14-r3::gentoo'
Comment 2 Guy 2013-08-28 01:53:56 UTC
As a temporary measure, I used this workaround:

LINGUAS="" emerge -1 =app-text/a2ps-4.14-r3

as I got confused while reading the above noted bug's comments.

# eix app-text/a2ps
[I] app-text/a2ps
     Available versions:  4.14-r1^t 4.14-r2^t 4.14-r3^t {emacs latex nls static-libs vanilla LINGUAS="ja" USERLAND="BSD GNU"}
     Installed versions:  4.14-r3^t(06:17:19 AM 08/16/2013)(nls -emacs -latex -static-libs -vanilla LINGUAS="-ja" USERLAND="GNU -BSD")
     Homepage:            http://www.inf.enst.fr/~demaille/a2ps/
     Description:         Any to PostScript filter
Comment 3 Alexandre Ferreira 2014-03-14 02:23:36 UTC
Same problem here. Found a patch and created an ebuild that applies that patch. It compiles with texinfo 5.x.
Comment 4 Alexandre Ferreira 2014-03-14 02:25:16 UTC
Created attachment 372630 [details, diff]
corrects compilation with texinfo 5.x

Patch extracted from https://abf.rosalinux.ru/openmandriva/a2ps/blob/master/a2ps-4.14-texinfo-5.x.patch
Comment 5 Alexandre Ferreira 2014-03-14 02:26:14 UTC
Created attachment 372632 [details]
ebuild that applies the textinfo patch

Applies the patch texinfo 5.x.
Comment 6 Nils Turner 2015-08-21 22:33:39 UTC
I can confirm that Alexandre's patch and modified ebuild allow it to install correctly.  Simply move into the "/usr/portage/app-text/a2ps" directory, add the patch (attachment 372630 [details, diff]) to the "files" directory (it should be named "a2ps-4.14-texinfo-5.x.patch"), replace "a2ps-4.14-r3.ebuild" with the modified ebuild (attachment 372632 [details]), remove the manifest, and rebuild the manifest with the command "ebuild a2ps-4.14-r3.ebuild manifest".
Comment 7 Nils Turner 2015-08-21 22:37:59 UTC
emerge as normal.  When the portage tree updates, these changes will be lost, so just apply them again until the tree is updated permanently.
Comment 8 Grant Edwards 2016-02-21 20:13:20 UTC
Apparently this is still broken after 3 years?  Using the above patch
and ebuild I was finally able to install a2ps.  I take it this is not
going to get fixed and I should just move the whole thing to my local
portage tree?
Comment 9 Andrew Savchenko gentoo-dev 2016-05-03 07:14:09 UTC
@printing and @cjk teams, I confirm that proposed patch works with the latest texinfo-6.1.

Hereby I propose (see r3 -> r4 diff below):
- update ebuild to EAPI 6;
- add missing die calls;
- include patch from comment 4;
- drop unused patch a2ps-4.13c-psset.patch from ${FILESDIR}.

I will apply these changes in two weeks if there is no objection (or earlier if you approve).
Comment 10 Andrew Savchenko gentoo-dev 2016-05-03 07:15:43 UTC
Created attachment 433014 [details, diff]
a2ps-4.14-r4.ebuild.diff

Patche with proposed ebuild changes.
Comment 11 Andrew Savchenko gentoo-dev 2016-05-03 07:18:50 UTC
Created attachment 433016 [details, diff]
a2ps-4.14-r4.ebuild.diff

Oops, forgot to dekeyword all arches.
Comment 12 Andrew Savchenko gentoo-dev 2016-05-17 18:31:51 UTC
Patch applied in tree, bumped to -r4.