[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800





--- Comment #11 from Akira TAGOH [EMAIL PROTECTED]  2008-10-15 02:07:26 EDT 
---
unable to build the package due to FAILED: BuildError: package ebnetd not in
list for tag dist-{f10,{f9,f8}-updates-candidate}.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800


Akira TAGOH [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #12 from Akira TAGOH [EMAIL PROTECTED]  2008-10-15 03:34:22 EDT 
---
there seems to be a delay. it's ok now.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800


Akira TAGOH [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #9 from Akira TAGOH [EMAIL PROTECTED]  2008-10-14 09:22:37 EDT ---
New Package CVS Request
===
Package Name: ebnetd
Short Description: EBNET protocol server
Owners: tagoh
Branches: devel F-9 F-8
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800





--- Comment #10 from Huzaifa S. Sidhpurwala [EMAIL PROTECTED]  2008-10-15 
01:25:46 EDT ---
cvs done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800


Huzaifa S. Sidhpurwala [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800


Jon Ciesla [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #8 from Jon Ciesla [EMAIL PROTECTED]  2008-10-13 09:03:19 EDT ---
Perfect.  Thanks!

APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800





--- Comment #7 from Akira TAGOH [EMAIL PROTECTED]  2008-10-12 03:49:13 EDT ---
Thanks. updated.

Spec URL: http://tagoh.fedorapeople.org/ebnetd/ebnetd.spec
SRPM URL: http://tagoh.fedorapeople.org/ebnetd/ebnetd-1.0-4.fc9.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800





--- Comment #5 from Akira TAGOH [EMAIL PROTECTED]  2008-10-11 08:12:42 EDT ---
Thanks for the review. I'd prefer adding a document how to change the port too.
updated srpm and spec file is:

Spec URL: http://tagoh.fedorapeople.org/ebnetd/ebnetd.spec
SRPM URL: http://tagoh.fedorapeople.org/ebnetd/ebnetd-1.0-3.fc9.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800


Patrice Dumas [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #6 from Patrice Dumas [EMAIL PROTECTED]  2008-10-11 10:47:45 EDT 
---
Please avoid using fedora in the file name:
http://fedoraproject.org/wiki/PackageMaintainers/Packaging_Tricks#Avoiding_using_fedora_or_redhat

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-10-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800


Jon Ciesla [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




--- Comment #4 from Jon Ciesla [EMAIL PROTECTED]  2008-10-10 10:41:06 EDT ---
What's comment #3 about?

Anyway. . .

rpmlint on SRPM is clean.

rpmlint on RPMS:

ebhttpd.i386: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.

ebnetd.i386: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.

ebnetd-common.i386: W: non-standard-uid /var/lib/ebnetd ebnetd
A file in this package is owned by a non standard user. Standard users are:
root, bin, daemon, adm, lp, sync, shutdown, halt, mail, news, uucp, operator,
games, gopher, ftp, nobody.

ebnetd-common.i386: W: non-standard-gid /var/lib/ebnetd ebnetd
A file in this package is owned by a non standard group. Standard groups are:
root, bin, daemon, sys, adm, tty, disk, lp, mem, kmem, wheel, mail, news,
uucp, man, games, gopher, dip, ftp, lock, nobody, users.

ebnetd-common.i386: W: non-standard-uid /var/run/ebnetd ebnetd
A file in this package is owned by a non standard user. Standard users are:
root, bin, daemon, adm, lp, sync, shutdown, halt, mail, news, uucp, operator,
games, gopher, ftp, nobody.

ebnetd-common.i386: W: non-standard-gid /var/run/ebnetd ebnetd
A file in this package is owned by a non standard group. Standard groups are:
root, bin, daemon, sys, adm, tty, disk, lp, mem, kmem, wheel, mail, news,
uucp, man, games, gopher, dip, ftp, lock, nobody, users.

ndtpd.i386: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.



These are all fine, see above.

Otherwise, everything looks great, my only other concern after a full review is
the Conflicts: for ebhttpd.  Will users expect this to be on port 80 by
default, period, full stop, or would it be feasible to set it to use something
else by default, or recommend that to users?  It'd be as simple as modifying
the xinetd file.  My preference would be to leave it as is, drop the
Conflicts:, but add a ebnetd-README.fedora indicating the port 80 issues and
resolutions, so that if someone wants to use this on a machine with an existing
httpd/tux/whatever, then they have that option and won't be stopped by the RPM.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-09-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=446800


Tony Fu [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Comment #3 from Tony Fu [EMAIL PROTECTED]  2008-09-09 23:07:22 EDT ---
requested by Jens Petersen (#27995)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-07-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ebnetd - EBNET protocol server


https://bugzilla.redhat.com/show_bug.cgi?id=446800





--- Additional Comments From [EMAIL PROTECTED]  2008-07-07 05:32 EST ---
Thanks, updated.

Spec URL: http://tagoh.fedorapeople.org/ebnetd/ebnetd.spec
SRPM URL: http://tagoh.fedorapeople.org/ebnetd/ebnetd-1.0-2.fc9.src.rpm


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446800] Review Request: ebnetd - EBNET protocol server

2008-07-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ebnetd - EBNET protocol server


https://bugzilla.redhat.com/show_bug.cgi?id=446800


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-07-04 17:17 EST ---
You'll need to remove the errant info dir in order for this to build.

You shouldn't include the same six documentation files in every subpackage.

That's all I see at a quick glance.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review