Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 38264 - Ebuild for seyon needed
Summary: Ebuild for seyon needed
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High enhancement (vote)
Assignee: Desktop Misc. Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-01-15 01:18 UTC by zhcnmccmzhdvfevooo
Modified: 2004-03-01 01:30 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
fixes some make install problems where it tries to install some unneeded configs (seyon-makefile-patch.patch,562 bytes, patch)
2004-02-10 01:03 UTC, James W. Dumay
Details | Diff
comments out some buggy test code (seyon-compile-fix.patch,1.70 KB, patch)
2004-02-10 01:04 UTC, James W. Dumay
Details | Diff
seyon-2.20c ebuild (seyon-2.20c.ebuild,1.11 KB, text/plain)
2004-02-10 01:05 UTC, James W. Dumay
Details
fixed makefile issue (seyon-2.20c.ebuild.patch,544 bytes, patch)
2004-02-10 02:59 UTC, James W. Dumay
Details | Diff
seyon-2.20c.ebuild (seyon-2.20c.ebuild,1.29 KB, text/plain)
2004-02-11 04:52 UTC, James W. Dumay
Details
seyon-2.20c-compile-fix.patch (seyon-2.20c-compile-fix.patch,1.70 KB, patch)
2004-02-11 04:52 UTC, James W. Dumay
Details | Diff
seyon-2.20c-lock-file-fix.patch (seyon-2.20c-lock-file-fix.patch,976 bytes, patch)
2004-02-11 04:53 UTC, James W. Dumay
Details | Diff
seyon-2.20c-makefile-patch.patch (seyon-2.20c-makefile-patch.patch,562 bytes, patch)
2004-02-11 04:54 UTC, James W. Dumay
Details | Diff
seyon-2.20c.lockdir.patch (seyon-2.20c.lockdir.patch,930 bytes, patch)
2004-02-11 05:05 UTC, James W. Dumay
Details | Diff
Seyon with even more fixed locks (seyon-2.20c-lock-file-fix.patch,1.21 KB, patch)
2004-02-13 00:09 UTC, James W. Dumay
Details | Diff
patch for updated locks on seyon ebuild (seyon-2.20c.ebuild-updated-locks.patch,942 bytes, patch)
2004-02-13 00:12 UTC, James W. Dumay
Details | Diff
seyon-2.20c-makefile-patch.patch (seyon-2.20c-makefile-patch.patch,562 bytes, patch)
2004-02-13 00:14 UTC, James W. Dumay
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description zhcnmccmzhdvfevooo 2004-01-15 01:18:04 UTC
I think there should really be a ebuild for seyon.
Its a great "Full-featured native X11 communications program."
Most Linux Distris have this in the builds.

Here is the Discription from Debian:
http://packages.debian.org/stable/comm/seyon.html
Comment 1 Marcelo Goes (RETIRED) gentoo-dev 2004-01-15 15:55:04 UTC
The most recent sources I found for this package are located at:
http://www.ibiblio.org/pub/Linux/apps/serialcomm/dialout/seyon-2.20c.tar.gz

even though documentation provided with the package suggested
ftp://sipb.mit.edu/pub/seyon/
as its central release location.

Assuming 2.20c is the most recent source version, being it released in May 9 1999, this is a rather old package.

This package would obviously need patching to work:

In file included from Seyon.c:25:
SeDecl.h:69: error: conflicting types for `usleep'
/usr/include/unistd.h:383: error: previous declaration of `usleep'
make: *** [Seyon.o] Error 1

But distributing a patch is not an option considering the package's license:
"Permission is also granted to modify the source as long as the modified source is not distributed."

Unless I am mistaken, I don't see how an ebuild+patch can be included without violating the copyright notice.
Comment 2 zhcnmccmzhdvfevooo 2004-01-16 07:06:00 UTC
Thats true that its an older package, but still one of the best tools for this usage.

The License: If you look in in the File  1-COPYING which is provided with the seyon-2.20c, you will see that the Author gave the ok to change the license to gpl. And I think with this, it is possible to distribute the patched sources.

---- CUT---
Seyon 2.20b and above are released under the GNU General Public
License by kind permission of the author and original maintainer of
seyon, Muhammad M. Saggaf. I contacted him by snail mail to ask him to
open up the code and here is his response:

======================================================================

I received your letter today (yes, I know it took a long time, it took a
trip half-way around the world and then back to my current address in the
U.S.). I share your view about license for Seyon, I think it is too
restrictive, especially that I'm not actively maintaining it. The purpose
behind that restriction was to prevent incompatible versions, really, but I
don't think it was the correct way of doing that. You have my permission to
modify the license (e.g. the GPL is just fine) and distribute the package
with the new license.

--- CUT END ---
Comment 3 Donnie Berkholz (RETIRED) gentoo-dev 2004-01-27 12:21:39 UTC
You guys want it?
Comment 4 zhcnmccmzhdvfevooo 2004-01-30 14:14:57 UTC
Sure I want it.
Thats the reason I opened that bug.
I already tried to create one, but I didn succeded...
Comment 5 Ithamar Garbe 2004-01-31 05:39:22 UTC
I would appreciate the new ebuild, too.
Comment 6 Gerhard Kusma 2004-02-02 01:37:20 UTC
I would also like to see an ebuilt of seyon soon. It is the best terminal programm i have seen.
Comment 7 James W. Dumay 2004-02-10 00:07:35 UTC
creating an ebuild for seyon now. gotten past some errors regarding some test caluses in the code (Just commented them out, they are not needed). Ill have an ebuild up soon.

Alexander Schneider, if you could, send me a copy of the email from the author so I can include it into the package.
Comment 8 James W. Dumay 2004-02-10 01:03:35 UTC
Created attachment 25320 [details, diff]
fixes some make install problems where it tries to install some unneeded configs
Comment 9 James W. Dumay 2004-02-10 01:04:51 UTC
Created attachment 25321 [details, diff]
comments out some buggy test code
Comment 10 James W. Dumay 2004-02-10 01:05:14 UTC
Created attachment 25322 [details]
seyon-2.20c ebuild
Comment 11 James W. Dumay 2004-02-10 01:05:42 UTC
ebuilds attached, pending approval.
Comment 12 Andrew Bevitt 2004-02-10 01:59:35 UTC
src_install needs 
make DESTDIR=${D} install.man 

Otherwise man pages dont get installed :)
Comment 13 James W. Dumay 2004-02-10 02:59:17 UTC
Created attachment 25329 [details, diff]
fixed makefile issue
Comment 14 Andrew Bevitt 2004-02-10 04:28:40 UTC
Just for notes sake, this compiles and installs on amd64 aswell...

I dont have any means or need to test the use though.
Comment 15 zhcnmccmzhdvfevooo 2004-02-10 09:56:25 UTC
James W. Dumay the email i talked of, is already in the package itself.
just look at the file 1-COPYING, which is in the .tar.gz file of seyon.
tnx for ebuild, I will also test them now.
Comment 16 zhcnmccmzhdvfevooo 2004-02-10 10:20:40 UTC
Compiling worked very fine.
But the program itself does not. This error occours after startup:

*** PASTE START ***
Locating Modems...
>> Error: Could not create temporary lock file /usr/spool/uucp/LTMP.8658: No such file or directory.
>> Error: Modem ``/dev/ttyS0'' is Locked.
Modem `/dev/ttyS0' is Unavailable.

>> Error: No Modems Available.
*** PASTE END***

This is because the /usr/spool/uucp directory does not exists. I think in the "#define LF_PATH" in config.h should get changed to /var/lock/uucp and a dependencie for taylor-uucp should be added. ( I dont know if uucp is really needed, but it creates the /var/lock/uucp directory).
I tested the ebuild on x86, so I dont know if these changes are also necessary for other platforms.
Comment 17 James W. Dumay 2004-02-11 04:52:00 UTC
Created attachment 25395 [details]
seyon-2.20c.ebuild

I didnt create a patch because it was looking abit confusing (too many files to
download and patch in order, etc)
Comment 18 James W. Dumay 2004-02-11 04:52:57 UTC
Created attachment 25396 [details, diff]
seyon-2.20c-compile-fix.patch

changed name as suggested by devs
Comment 19 James W. Dumay 2004-02-11 04:53:37 UTC
Created attachment 25397 [details, diff]
seyon-2.20c-lock-file-fix.patch

should fix lockfile problem
Comment 20 James W. Dumay 2004-02-11 04:54:05 UTC
Created attachment 25399 [details, diff]
seyon-2.20c-makefile-patch.patch

changed name as suggested by devs
Comment 21 James W. Dumay 2004-02-11 05:05:04 UTC
Created attachment 25401 [details, diff]
seyon-2.20c.lockdir.patch

I must be on something, I tells ya...
Comment 22 zhcnmccmzhdvfevooo 2004-02-11 09:32:52 UTC
no problems with the new ebuild on my testmachine. works very good.
But why do you do this in the new ebuild:

touch /etc/lock

I really see no reason for this...
Comment 23 James W. Dumay 2004-02-12 21:50:04 UTC
I just wasnt sure. It uses this file to record locks.
Comment 24 James W. Dumay 2004-02-13 00:09:19 UTC
Created attachment 25518 [details, diff]
Seyon with even more fixed locks

<seemant> i386: suggestion -- make it put the locks in /var/lock somewhere
instead of /etc/loc
Comment 25 James W. Dumay 2004-02-13 00:12:38 UTC
Created attachment 25519 [details, diff]
patch for updated locks on seyon ebuild
Comment 26 James W. Dumay 2004-02-13 00:14:16 UTC
Created attachment 25520 [details, diff]
seyon-2.20c-makefile-patch.patch

updated locks
Comment 27 James W. Dumay 2004-02-13 00:14:37 UTC
This ebuild is quite mature as of now.
Comment 28 Brandon Hale (RETIRED) gentoo-dev 2004-02-28 06:47:24 UTC
phew, all finished here?
cyfred, feel free to commit if youd like, there seems to be a fair demand here.
if not, I'll get back to sorting through all this sometime.
james, have you filed any of this upstream? it looks like their packages is fairly b0rked if you have to hack around so much.
Comment 29 Andrew Bevitt 2004-02-28 15:05:28 UTC
Probable category choice : x11-misc ???

Anyone have a problem with that or think something else would be better, let us know soon and we'll get this into the tree.
Comment 30 Brandon Hale (RETIRED) gentoo-dev 2004-02-29 06:23:01 UTC
x11-misc WORKSFORME :)
Comment 31 James W. Dumay 2004-02-29 12:52:56 UTC
Haha. Im cool with x11-misc.
Comment 32 Andrew Bevitt 2004-03-01 01:30:37 UTC
OK peoples just went into CVS so should be out on the mirrors in an hour. I didnt change much from here except cleaned up the ebuild a bit to remove whitespaces. 

Marking as FIXED