This is the mail archive of the cygwin-apps mailing list for the Cygwin project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Hi Daniel, On Feb 13 07:56, D. Boland wrote: > Hi Corinna, > > Corinna Vinschen wrote: > >On Feb 7 18:55, D. Boland wrote: > >>Some programs in the inetutils suite are packaged already: > >> > >>* hostname > >>* whois > >>* tftp > >>* tftp-server > >> > >>So I added these on the 'required' lines. > > > >They are not actually *required* to run inetd, right? Does it really > >make sense to add them as require packages then? > > They belong to the package. A user expects them in there and would have > to piece the package back together. Why does it make sense to leave them > out? These tools are provided separately in many Linux distros for quite some time, and while those tools can be started by inetd, inetd doesn't require them and they don't require inetd (xinetd is perfectly capable of replacing inetd). > >>Also, the following include files are needed: > >> > >>* /usr/include/netinet/icmp6.h > >>* /usr/include/netinet/ip_var.h > >> > >>There has been discussion on including the icmp6.h file in Cygwin 2.3.0. > > > >Can you check if icmp6.h from FreeBSD > >https://svnweb.freebsd.org/base/head/sys/netinet/icmp6.h?revision=279531&view=co > >does the trick when copied to /usr/include/netinet? If so, I add this file > >to the repo. > > If I use the FreeBSD version I get: > > gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I.. -I../lib -I../lib -I../libinetutils > -I../libicmp -O2 -pipe -Wimplicit-function-declaration -MT ping6.o -MD > -MP -MF .deps/ping6.Tpo -c -o ping6.o ping6.c > ping6.c: In function 'ping_init': > ping6.c:844:37: error: 'ICMP6_FILTER' undeclared (first use in this > function) > setsockopt (fd, IPPROTO_ICMPV6, ICMP6_FILTER, &filter, sizeof > (filter)); That won't work. Ever. Using RAW sockets requires administrator privileges. Apart from that, WinSock setsockopt does not at all support settings for the IPPROTO_ICMPV6 protocol family. That's what the 32 bit ping package I took a (not so) quick glance over the binary packages and there are some problems: - etc/defaults/etc/init.src.sh has 000 perms. - etc/defaults/etc/shells collides with the already provided file from the base-files package. - usr/bin/inetutils-server-config installs inetd and syslogd in one go. That's a no no. There should be two installation scripts since you can't expect that a user who wants one service also wants the other one. Some people would probably like to stick to the Windows logging, or install syslog-ng. - Apropos syslog-ng: syslogd potentially collides with syslog-ng. However, instead of reusing the existing /usr/bin/syslogd-config script, your new scripts don't check for an existing syslog-ng installation at all. - You removed the etc/defaults/etc/xinetd.d files. - You removed the /usr/bin/iu-config file, which was specificially created to allow running under the cyg_server account for the known reasons of being able to switch the user context(*). Your scripts enforce file ownership and service start under the SYSTEM account. That's why the csih package exists for service installation scripts to use for stuff like that. Please have a look into the old /usr/bin/iu-config file. - usr/bin/ping collides with the ping package. - sbin/ifconfig is mostly non-functional since Cygwin doesn't support most of the functionality. Do you really want to maintain it? - usr/bin/traceroute is non-functional: $ traceroute.exe www.wdr.de traceroute to e2636.g.akamaiedge.net (104.90.150.230), 64 hops max traceroute: socket: Operation not permitted - What also irritates me is that almost none of the patches from the former package made it into your version. Did you actually check the patches from the current 1.9.1 source package and made sure that they are really not required anymore, especially concerning O_BINARY/O_TEXT mode, authentication, exception handling, and, generally, backward compatibility? Corinna (*) https://cygwin.com/cygwin-ug-net/ntsec.html -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat
Attachment:
signature.asc
Description: PGP signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |