Opened 11 years ago
Closed 11 years ago
#5112 closed enhancement (fixed)
xscreensaver-5.27
Reported by: | Fernando de Oliveira | Owned by: | Fernando de Oliveira |
---|---|---|---|
Priority: | normal | Milestone: | 7.6 |
Component: | BOOK | Version: | SVN |
Severity: | normal | Keywords: | |
Cc: |
Change History (8)
comment:1 by , 11 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
follow-up: 3 comment:2 by , 11 years ago
by , 11 years ago
Attachment: | xscreensaver-5.27-comment_style-1.patch added |
---|
patcch to fix comments syntax
comment:3 by , 11 years ago
First a problem with phtread, they decided to include, don't know how, solved with LIBS=-pthread .configure ... (error log at the end, below).
LIBS=-lpthread ./configure ...
comment:4 by , 11 years ago
I don't know why you need the patch. gcc certainly understands in .c code.
The install fails because it want to install suid root. You have to be root to run install, even for DESTDIR installs.
comment:5 by , 11 years ago
Thank you very much, Bruce. I am trying to understand where it is trying to set uid, so I can change at least for the DESTDIR (don't like to use as root until I know where it is going).
Meanwhile, just sending the failure without the patch. I tis really necessary. I don't remember the few I read in C, but rceall comments with double bar, as you said.
gcc -pedantic -Wall -Wstrict-prototypes -Wnested-externs -Wmissing-prototypes \ -Wno-overlength-strings -Wdeclaration-after-statement -std=c89 \ -U__STRICT_ANSI__ -c -I. -I. -I./../utils -I.. -pthread \ -I/usr/include/libglade-2.0 -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include \ -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -I/usr/include/cairo \ -I/usr/include/pixman-1 -I/usr/include/pango-1.0 -I/usr/include/freetype2 \ -I/usr/include/libpng16 -I/usr/include/harfbuzz -I/usr/include/freetype2 \ -I/usr/include/libpng16 -I/usr/include/harfbuzz -I/usr/include/libxml2 \ -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng16 -I/usr/include/glib-2.0 \ -I/usr/lib/glib-2.0/include -pthread -I/usr/include/gdk-pixbuf-2.0 \ -I/usr/include/libpng16 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include \ -DSTANDALONE -DHAVE_CONFIG_H -g -O2 delaunay.c delaunay.c: In function ‘circumcircle’: delaunay.c:90:3: error: expected expression before ‘/’ token // Original ^ delaunay.c:92:3: error: expected expression before ‘/’ token // Proposed by Chuck Morris ^ delaunay.c:47:16: warning: variable ‘drsqr’ set but not used [-Wunused-but-set-variable] double dx,dy,drsqr; ^ delaunay.c:94:1: warning: control reaches end of non-void function [-Wreturn-type] } ^ Makefile:531: recipe for target 'delaunay.o' failed make[1]: *** [delaunay.o] Error 1 make[1]: Leaving directory \ '/home/fernando/tmp/paco-build-2014.05.29-20h13m42s/xscreensaver-5.27/hacks' Makefile:26: recipe for target 'default' failed make: *** [default] Error 5
I should have included a question mark (syntax error?).
comment:6 by , 11 years ago
Tried:
sudo make install_prefix=/home/fernando/tmp/paco-build-2014.05.29-20h31m59s/DEST-xscreensaver-5.27 install
and got exactly the same error
Finally, ran the entire build as root and it installed some parts untill the same error.
comment:7 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Bruce, thank you very much for your help!
Fixed at r13190.
I am having a hell of problems with this.
First a problem with phtread, they decided to include, don't know how, solved with LIBS=-pthread .configure ... (error log at the end, below).
Second a problem with comments written in a wrong way (wrong syntax, I think), solved with a patch from gentoo. Will attach it after sending this comment.
Now, I cannot understand, when trying to make install_prefix=$DESTINODIR install, it errors with:
Below, as mentioned above, the log for the first problem, solved.