Hi Martín (2010.06.29_06:01:00_+0200) > I' m not too sure about the gcc patch. It doesn't seem to do anything > useful, and I don't see any gcc warnings when compiling main... What > was the rationale for this?
I don't know why Ubuntu's gcc is spewing out more warnings, but it is: gcc -Wall -g -O2 -I/usr/include/lzo -I/usr/include/openssl -I/usr/include/openssl -I/usr/include/openssl -I/usr/include/openssl -DVTUN_CONFIG_FILE=\"/etc/vtund.conf\" -DVTUN_PID_DIR=\"/var/run\" -DVTUN_STAT_DIR=\"/var/log/vtund\" -DVTUN_LOCK_DIR=\"/var/lock/vtund\" -c main.c main.c: In function ‘main’: main.c:194: warning: ignoring return value of ‘dup’, declared with attribute warn_unused_result main.c:195: warning: ignoring return value of ‘dup’, declared with attribute warn_unused_result main.c:196: warning: ignoring return value of ‘dup’, declared with attribute warn_unused_result main.c:201: warning: ignoring return value of ‘chdir’, declared with attribute warn_unused_result etc. However, that patch has no benefit and just hides warnings. We should be able to drop it in Ubuntu now that we have no need to fork the packaging any more. Thanks for the quick response, SR -- Stefano Rivera http://tumbleweed.org.za/ H: +27 21 465 6908 C: +27 72 419 8559 UCT: x3127
signature.asc
Description: Digital signature
