Re: Bug#919413: cascade of FTBFS

2019-02-14 Thread Andreas Tille
On Thu, Feb 14, 2019 at 03:16:22PM +0100, Dominique Dumont wrote:
> On Tuesday, 12 February 2019 16:54:12 CET Andreas Tille wrote:
> > I'm
> > not sure how to deal with the jquery.js one since this is potentially an
> > issue with lots of dependencies - I remember discussions about this
> > which I did not followed.
> 
> Fortunately, jquery is available as a Debian package.

Sure it is.  I simply remember some discussions about why doxygen needs its
own jquery.  I'd be really happy if this is not the case any more.
 
Kind regards

   Andreas.

-- 
http://fam-tille.de



Bug#919413: cascade of FTBFS

2019-02-14 Thread Paolo Greppi

Hi Dominique, see below

Il 14/02/2019 15:16, Dominique Dumont ha scritto:

On Tuesday, 12 February 2019 16:54:12 CET Andreas Tille wrote:

I'm
not sure how to deal with the jquery.js one since this is potentially an
issue with lots of dependencies - I remember discussions about this
which I did not followed.

Fortunately, jquery is available as a Debian package.

We had a similar issue with libmojolicious-perl. This package now:
- removes jquery from source tarball [1] using debian/copyright Files-excluded 
parameter
- depends on libjs-jquery
- provides a symlink to Debian's query instead of the regular jquery file using
   debian/libmojolicious-perl.links file [2]

HTH

[1] 
https://salsa.debian.org/perl-team/modules/packages/libmojolicious-perl/blob/master/debian/copyright#L7
[2] 
https://salsa.debian.org/perl-team/modules/packages/libmojolicious-perl/blob/master/debian/libmojolicious-perl.links


I 'm afraid we will not be able to avoid embedding jquery in doxygen, 
because it makes a weird use of it.

The matter has been nicely put down by the former maintaners, see:
https://salsa.debian.org/paolog-guest/doxygen/blob/master/debian/README.jquery

Paolo



Re: Bug#919413: cascade of FTBFS

2019-02-14 Thread Dominique Dumont
On Tuesday, 12 February 2019 16:54:12 CET Andreas Tille wrote:
> I'm
> not sure how to deal with the jquery.js one since this is potentially an
> issue with lots of dependencies - I remember discussions about this
> which I did not followed.

Fortunately, jquery is available as a Debian package.

We had a similar issue with libmojolicious-perl. This package now:
- removes jquery from source tarball [1] using debian/copyright Files-excluded 
parameter
- depends on libjs-jquery
- provides a symlink to Debian's query instead of the regular jquery file using 
  debian/libmojolicious-perl.links file [2]



HTH

[1] 
https://salsa.debian.org/perl-team/modules/packages/libmojolicious-perl/blob/master/debian/copyright#L7
[2] 
https://salsa.debian.org/perl-team/modules/packages/libmojolicious-perl/blob/master/debian/libmojolicious-perl.links




Bug#919413: cascade of FTBFS

2019-02-12 Thread Andreas Tille
Control: tags -1 help

Hi Paolo,

in my attempt to see what I can do for #921779 which breaks several
packages I stumbled upon your attempt to adopt doxygen.  Thanks a lot
for this brave intention. ;-)

I realised that the watch file did not work properly - feel free to `git
am` the attached patch.  I also noticed that there are remaining lintian
errors:

E: doxygen source: source-is-missing templates/html/jquery.js line length is 
32401 characters (>512)
N: 
N:The source of the following file is missing. Lintian checked a few
N:possible paths to find the source, and did not find it.
N:
N:Please repack your package to include the source or add it to
N:"debian/missing-sources" directory.
N:
N:If this is a false-positive, please report a bug against Lintian.
N:
N:Please note, that insane-line-length-in-source-file tagged files are
N:likely tagged source-is-missing. It is a feature not a bug.
N:
N:Severity: serious, Certainty: possible
N:
N:Check: cruft, Type: source
N: 
E: doxygen source: source-is-missing templates/html/menu.js line length is 695 
characters (>512)
E: doxygen source: source-is-missing templates/html/svgpan.js line length is 
312 characters (>256)


You should override the latter two since these are false positives.  I'm
not sure how to deal with the jquery.js one since this is potentially an
issue with lots of dependencies - I remember discussions about this
which I did not followed.

Regarding the ratt results[1] the issue

   librostlab: ! LaTeX Error: File 'listofitems.sty' not found.

can be solved by a doxygen-latex Build-Depends - at least I added this
to frobby in Git[2] which helped against this very error but after this it
was running into:

[572]
! Undefined control sequence.
l.211 ...+t-1$. The inner slice will have $(a\text
  {'},b)$, where $a^\prime$ ...

?.
! Emergency stop.
l.211 ...+t-1$. The inner slice will have $(a\text
  {'},b)$, where $a^\prime$ ...

!  ==> Fatal error occurred, no output PDF file produced!



I have no idea how to fix this.

Kind regards

   Andreas.



[1] https://salsa.debian.org/paolog-guest/doxygen/wikis/ratt4
[2] 
https://salsa.debian.org/science-team/frobby/commit/d2fd89875e8491a755ee702e710aa9e003982ee7

-- 
http://fam-tille.de
>From b0a8a6a14c391fbc40489ab6df984435efaba1c4 Mon Sep 17 00:00:00 2001
From: Andreas Tille 
Date: Tue, 12 Feb 2019 16:02:57 +0100
Subject: [PATCH] Fix watch file

---
 debian/changelog | 4 
 debian/watch | 4 ++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index beb9ad6..6ca0d0b 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,5 +1,6 @@
 doxygen (1.8.15-1) unstable; urgency=medium
 
+  [ Paolo Greppi ]
   * doxygen 1.8.15 release. Closes: #920447.
   * Do not produce "Directory Reference" man pages. Closes: #742871.
   * Bump debhelper compat.
@@ -13,6 +14,9 @@ doxygen (1.8.15-1) unstable; urgency=medium
   * Switch to llvm-toolchain-7. Closes: #912799.
   * Make the output of $year reproducible. Closes: #863054
 
+  [ Andreas Tille ]
+  * Fix watch file
+
  -- Paolo Greppi   Tue, 05 Feb 2019 16:45:02 +0100
 
 doxygen (1.8.13-10) unstable; urgency=medium
diff --git a/debian/watch b/debian/watch
index f688d00..2e294c4 100644
--- a/debian/watch
+++ b/debian/watch
@@ -1,4 +1,4 @@
-version=3
+version=4
 
-opts=filenamemangle=s/.+\/Release_(\d\S*)\.tar\.gz/doxygen-$1.tar\.gz/ \
+opts=uversionmangle=s/_/./g,filenamemangle=s/.+\/Release_(\d\S*)\.tar\.gz/doxygen-$1.tar\.gz/ \
   https://github.com/doxygen/doxygen/tags .*/Release_(\d\S*)\.tar\.gz
-- 
2.20.1