<tt><font size=2>Source: nanopolish</font></tt><br><tt><font size=2>Version: 0.13.2-1</font></tt><br><tt><font size=2>Severity: normal</font></tt><br><tt><font size=2>Tags: ftbfs<br>Justification: fails to build from source (but built successfully in the
past) on arch ppc64el</font></tt><br><tt><font size=2> </font></tt><br><tt><font size=2>-- System Information:</font></tt><br><tt><font size=2>Debian Release: bullseye/sid</font></tt><br><tt><font size=2>  APT prefers unstable</font></tt><br><tt><font size=2>  APT policy: (500, 'unstable')</font></tt><br><tt><font size=2>Architecture: ppc64el (ppc64le)</font></tt><br><tt><font size=2>Kernel: Linux 5.4.0-3-powerpc64le (SMP w/80 CPU cores)</font></tt><br><br><tt><font size=2>nanopolish FTBFS on ppc64el:</font></tt><br><a href="https://buildd.debian.org/status/fetch.php?pkg=nanopolish&arch=ppc64el&ver=0.13.2-1&stamp=1588529870&raw=0"><font size=3 color=blue><u>https://buildd.debian.org/status/fetch.php?pkg=nanopolish&arch=ppc64el&ver=0.13.2-1&stamp=1588529870&raw=0</u></font></a><font size=3></font><br><br><br><tt><font size=2>nanopolish-0.13.2/src/nanopolish_squiggle_read.h contains
the following enumeration:</font></tt><br><tt><font size=2> </font></tt><br><tt><font size=2>enum PoreType</font></tt><br><tt><font size=2>{</font></tt><br><tt><font size=2>    PT_UNKNOWN = 0,</font></tt><br><tt><font size=2>    PT_R7,</font></tt><br><tt><font size=2>    PT_R9,</font></tt><br><tt><font size=2>};</font></tt><br><tt><font size=2> </font></tt><br><tt><font size=2>Enumerators PT_R7 and PT_R9 are both already defined
elsewhere, which conflicts with this package and causes FTBFS in ppc64el.
 (I'm not sure where PT_R7 and PT_R9 are already defined.  A
quick grep of kernel source shows several in /usr/src/linux-xx/arch as
well as /usr/lib.)</font></tt><br><tt><font size=2> </font></tt><br><tt><font size=2>I verified that changing those names (PT_R7 and PT_R9)
in the enumeration and corresponding code fixes the problem in ppc64el
and allows it to build successfully.  Based on the build log files
for ppc64 and powerpc, this change should fix FTBFS for those as well.
</font></tt><br><tt><font size=2> </font></tt><br><tt><font size=2>I did not include a patch.  Since this problem
could be fixed in several different ways, including the possibility of
changing enumerator names, the maintainer might want it to match already
established conventions.</font></tt><br><tt><font size=2> </font></tt><br><tt><font size=2>A minor nit, removing the comma after PT_R9 would
make the code a little more standard and match the rest of the enums in
that .h file.</font></tt><br><tt><font size=2> </font></tt><br><tt><font size=2>Barry Arndt</font></tt><br><tt><font size=2> </font></tt><BR>