Spec Url: http://testapp1.iesabroad.org/~mmcgrath/ytalk/ytalk.spec SRPM Url: http://testapp1.iesabroad.org/~mmcgrath/ytalk/ytalk-3.3.0-2.src.rpm Description: The YTalk program is essentially a chat program for multiple users. YTalk works just like the UNIX talk program and even communicates with the same talk daemon(s), but YTalk allows for multiple connections (unlike UNIX talk). YTalk also supports redirection of program output to other users as well as an easy-to-use menu of commands. Misc: I was sad to see ytalk go in FC4 so hopefully we'll be able to get it into Extras. My main concern with this is the License. For some reason it changed from BSD to GPL. This just sounds odd to me (From the bottom of the INSTALL file): --------------------------------------- Most of these installation instructions are taken word-for-word from the `INSTALL' file that comes with GNU Wget. This means that, unlike the rest of the YTalk package (which is freely copyable as long as the copyright notices are kept in the sources), this file is (probably) covered by the GNU General Public License version 2. ---------------------------------------- I'm no license guru so I'm not sure if any GPL/BSD violations are going on: maintainer website: http://www.impul.se/ytalk/
Doh! You beat me to this one. I was going to contribute ytalk. Oh well. I snooze I lose. One thing, though. As of 3.3.0, ytalk no longer supports X. The developer's site says "at the request of many users." I'd sure like to know what is wrong with those users because in my opinion, the X support was the most important feature of ytalk. Now it's all but useless to me. :( So, as I was saying, I had plans to package ytalk, but didn't get to it in time. I was considering version 3.2.0 since it still supports X. Would you consider packaging that release instead, to include X support?
I thought it was odd too that users requested the support to be removed. Normally I'd think users would simply not use a feature they didn't want. My main concern with using 3.2.0 is that if the upstream has further releases (security or otherwise) it will become more and more difficult to support the package. It's also possible there is another maintainer of ytalk (it seems to have jumped around quite a bit) If we could find another maintainer that actively maintains ytalk and still supports X I'd be happy to use them as an upstream instead of www.impul.se
I forgot to mention earlier that I do not currently have a sponsor.
Greetings. I was going to do a review of this package, but the URL(s) in the submission appear to not be functional. testapp1.iesabroad.org doesn't resolve here. Can you update the ulrs and then I will be happy to do a review.
I forgot to update this link. The new locations are at: SRPM: http://mmcgrath.net/~mmcgrath/ytalk/ytalk-3.3.0-2.src.rpm SPEC: http://mmcgrath.net/~mmcgrath/ytalk/ytalk.spec Also I've been sponsored since posting this bug.
Greetings, heres a review: MUST items: OK - package name good. OK - license ok. (GPL) OK - spec file matches. OK - spec in english. OK - spec legible. OK - md5sum matches: c043a8d854638b293a3b645d8600aa38 ytalk-3.3.0.tar.gz c043a8d854638b293a3b645d8600aa38 ytalk-3.3.0.tar.gz.1 OK - files and dirs ok. OK - clean section good. OK - macros good. OK - builds ok in fc4. OK - compiles and builds under devel. OK - builds ok in mock on devel. OK - rpmlint has no output. Minor/non blockers: - Might include COPYING, README, AUTHORS, Changelog as docs? APPROVED.
I've added the doc's. Thanks for the review.