John Gnew wrote these words on 03/26/05 15:27 CST: > I have attached a patch file that will change xinetd.conf to include > xinetd.d/ directory. > > This also included updating CVSserver, Subversion server, Fam, Qpopper, > Samba, and Leafnode. > > Could someone create this into an html file and tell me how it looks. I > have not yet figured out how to view these files other than as XML in an > editor.
Overall, a very fine job. Especially considering you didn't use the XML tools and validate the XML. It validates and renders clean. Good job! Archaic pointed out some things. Especially the whitespace thing. Once you get the XML tools setup and you render, you'd have noticed this. No big deal. Some other notes. 1. In at least one spot, you didn't change the "adding an entry" to "create the file" when referencing the addition of an inet.d file. 2. Why do you end a command with closing [/command] tags, simply to open the very next line with an opening [command tag]? Both can be omitted. I hope this makes sense what I'm saying. 3. In an install command, the -o root and -g root parameters are redundant. Root will be issuing the command and the default is to use root's UID and GID. -- Randy rmlscsi: [GNU ld version 2.15.94.0.2 20041220] [gcc (GCC) 3.4.3] [GNU C Library stable release version 2.3.4] [Linux 2.6.10 i686] 18:23:00 up 24 days, 4:27, 5 users, load average: 0.24, 0.76, 0.70 -- http://linuxfromscratch.org/mailman/listinfo/blfs-dev FAQ: http://www.linuxfromscratch.org/blfs/faq.html Unsubscribe: See the above information page
