Bug#880997: nageru: FTBFS on armel

2017-11-07 Thread Adrian Bunk
On Tue, Nov 07, 2017 at 12:48:03PM +0100, Steinar H. Gunderson wrote:
> On Tue, Nov 07, 2017 at 12:21:24PM +0200, Adrian Bunk wrote:
> >> FWIW, last time I asked about this, it was a toolchain bug on armel
> >> (std::atomic doesn't work). Is there any other information I should know
> >> about?
> > It is not true that std::atomic doesn't work on armel,
> > and that's not what the error message is about.
> 
> If you can't use something from std:: without linking to an extra library,
> it “doesn't work” in my world. :-) Especially given that it used to work just
> fine.

armel needing -latomic here is new with gcc 7,
but it already used to not work on mips/mipsel
and several non-release architectures before.

> > The following patch fixes the FTBFS on armel
> > and the other architectures where nageru currently FTBFS:
> 
> I'll be happy to include the patch in the next upload, but I do consider it a
> workaround (and in particular, I won't be pulling it upstream).

My patch is for debian/rules and points to the gcc bug.

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed



Bug#880997: nageru: FTBFS on armel

2017-11-07 Thread Steinar H. Gunderson
On Tue, Nov 07, 2017 at 12:21:24PM +0200, Adrian Bunk wrote:
>> FWIW, last time I asked about this, it was a toolchain bug on armel
>> (std::atomic doesn't work). Is there any other information I should know
>> about?
> It is not true that std::atomic doesn't work on armel,
> and that's not what the error message is about.

If you can't use something from std:: without linking to an extra library,
it “doesn't work” in my world. :-) Especially given that it used to work just
fine.

> The following patch fixes the FTBFS on armel
> and the other architectures where nageru currently FTBFS:

I'll be happy to include the patch in the next upload, but I do consider it a
workaround (and in particular, I won't be pulling it upstream).

/* Steinar */
-- 
Homepage: https://www.sesse.net/



Bug#880997: nageru: FTBFS on armel

2017-11-07 Thread Adrian Bunk
Control: tags -1 patch

On Mon, Nov 06, 2017 at 08:59:37PM +0100, Steinar H. Gunderson wrote:
> On Mon, Nov 06, 2017 at 08:51:27PM +0100, Sebastian Ramacher wrote:
> > | /usr/bin/ld: httpd.o: undefined reference to symbol 
> > '__atomic_fetch_sub_8@@LIBATOMIC_1.0'
> > | //usr/lib/arm-linux-gnueabi/libatomic.so.1: error adding symbols: DSO 
> > missing from command line
> 
> FWIW, last time I asked about this, it was a toolchain bug on armel
> (std::atomic doesn't work). Is there any other information I should know
> about?

It is not true that std::atomic doesn't work on armel,
and that's not what the error message is about.

The following patch fixes the FTBFS on armel
and the other architectures where nageru currently FTBFS:

--- debian/rules.old2017-11-07 09:43:20.849529546 +
+++ debian/rules2017-11-07 09:43:20.849529546 +
@@ -1,5 +1,8 @@
 #!/usr/bin/make -f
 
+# see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81358
+export DEB_LDFLAGS_MAINT_APPEND = -Wl,--as-needed -latomic
+
 export QT_SELECT=5
 
 %:


cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed



Bug#880997: nageru: FTBFS on armel

2017-11-06 Thread Steinar H. Gunderson
On Mon, Nov 06, 2017 at 08:51:27PM +0100, Sebastian Ramacher wrote:
> | /usr/bin/ld: httpd.o: undefined reference to symbol 
> '__atomic_fetch_sub_8@@LIBATOMIC_1.0'
> | //usr/lib/arm-linux-gnueabi/libatomic.so.1: error adding symbols: DSO 
> missing from command line

FWIW, last time I asked about this, it was a toolchain bug on armel
(std::atomic doesn't work). Is there any other information I should know
about?

/* Steinar */
-- 
Homepage: https://www.sesse.net/



Bug#880997: nageru: FTBFS on armel

2017-11-06 Thread Sebastian Ramacher
Source: nageru
Version: 1.6.2-1
Severity: serious
Justification: fails to build from source (but built successfully in the past)
Tags: sid buster
Control: block 880355 by -1

nageru FTBFS on armel when binNMUed for the libva transition:
| g++ -o nageru glwidget.o mainwindow.o vumeter.o lrameter.o 
compression_reduction_meter.o correlation_meter.o aboutdialog.o analyzer.o 
input_mapping_dialog.o midi_mapping_dialog.o nonlinear_fader.o glwidget.moc.o 
mainwindow.moc.o vumeter.moc.o lrameter.moc.o compression_reduction_meter.moc.o 
correlation_meter.moc.o aboutdialog.moc.o analyzer.moc.o 
input_mapping_dialog.moc.o midi_mapping_dialog.moc.o nonlinear_fader.moc.o 
ellipsis_label.moc.o clickable_label.moc.o context_menus.o vu_common.o 
piecewise_interpolator.o main.o midi_mapper.o midi_mapping.pb.o 
chroma_subsampler.o v210_converter.o mixer.o basic_stats.o metrics.o 
pbo_frame_allocator.o context.o ref_counted_frame.o theme.o httpd.o flags.o 
image_input.o alsa_output.o disk_space_estimator.o print_latency.o 
timecode_renderer.o tweaked_inputs.o audio_mixer.o alsa_input.o alsa_pool.o 
ebu_r128_proc.o stereocompressor.o resampling_queue.o correlation_measurer.o 
filter.o input_mapping.o state.pb.o quicksync_encoder.o x264_encoder.o 
x264_dynamic.o x264_speed_control.o video_encoder.o metacube2.o mux.o 
audio_encoder.o ffmpeg_raii.o ffmpeg_util.o decklink_capture.o decklink_util.o 
decklink_output.o decklink/DeckLinkAPIDispatch.o ffmpeg_capture.o -Wl,-z,relro 
-lQt5OpenGLExtensions -lQt5OpenGL -lQt5PrintSupport -lQt5Widgets -lQt5Gui 
-lQt5Core -lusb-1.0 -lmovit -llua5.2 -lmicrohttpd -lepoxy -lx264 -lprotobuf 
-pthread -lpthread -lpci -lbmusb -pthread -lva -lva-drm -lva-x11 -lX11 
-lavformat -lavcodec -lavutil -lswscale -lavresample -lzita-resampler -lasound 
-ldl -lqcustomplot
| /usr/bin/ld: httpd.o: undefined reference to symbol 
'__atomic_fetch_sub_8@@LIBATOMIC_1.0'
| //usr/lib/arm-linux-gnueabi/libatomic.so.1: error adding symbols: DSO missing 
from command line

See
https://buildd.debian.org/status/fetch.php?pkg=nageru=armel=1.6.2-1+b1=1509844496=0
for the full log.

Cheers
-- 
Sebastian Ramacher


signature.asc
Description: PGP signature