Bug#635382: texlive-latex-extra: ucshyper check for PDF mode buggy and not necessary anyway

[Lionel Elie Mamane <lionel@mamane.lu>]

On Mon, Jul 25, 2011 at 06:35:15PM +0200, Lionel Elie Mamane wrote:
> Package: texlive-latex-extra
> Tags: patch

> 3) ucshyper does not actually need the output mode to be PDF, nor the
>    TeX implementation to be pdfTeX. I just _removed_ the above test,
>    as well as the following line:

> \RequirePackage[pdftex,unicode]{hyperref}

> So possibly this test should be changed to something like:

> \ifdefined\PrerenderUnicode\else%
> \PackageWarning{ucshyper}{ucshyper needs to be run with
>     inputenc utf8x.\MessageBreak Disabling functionality}%
>   \expandafter\endinput\fi

> _and_ still remove the

> \RequirePackage[pdftex,unicode]{hyperref}

> line so that users can choose their own hyperref driver freely. Maybe
> add instead:

> \ifdefined\hypersetup%
>  \hypersetup{unicode=true}%
> \else%
>   \PackageWarning{ucshyper}{ucshyper needs to be loaded after
>       hyperref.\MessageBreak Disabling functionality}%
>     \expandafter\endinput%
> \fi%

I downloaded the texlive-extra (Debian) source package; it contains
ucshyper.sty, which says it is generated from ucs.dtx, but does not
contain ucs.dtx. So

1) I cannot prepare a "real proper" patch.

2) This looks like an RC bug to me, not shipping the preferred form
   for modification in the source package.

Additionally, browsing the upstream SVN, I found neither ucs.dtx, nor
ucshyper.sty, but probably I just didn't look in the right place.

-- 
Lionel