Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=339631
Summary: Review Request: tcputils - Utilities for TCP programming in shell-scripts Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: allisson@gmail.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://allisson.fedorapeople.org/packages/tcputils/tcputils.spec SRPM URL: http://allisson.fedorapeople.org/packages/tcputils/tcputils-0.6.2-1.fc7.src....
Description: This is a collection of programs to facilitate TCP programming in shell-scripts. There is also a small library which makes it somewhat easier to create TCP/IP sockets.
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |mtasaka@ioa.s.u-tokyo.ac.jp Status|NEW |ASSIGNED Flag| |fedora-review?
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-10-19 08:44 EST ------- Allisson, I would appreciate it if you would review my review request bug 336401 and bug 336421 . (bug 336421 depends on bug 336401.
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-10-19 10:29 EST ------- For 0.6.2-1
* Timestamps - For this package, ------------------------------------------------------ make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p" ------------------------------------------------------ works to keep timestamps on man files. For this package it is really preferable because these man files are written more than 10 years ago.
- Also you may want to keep timestamps on the tarball in your srpm. You can use "wget -N", for example.
! %attr ------------------------------------------------------- %attr(0644, root, root) %{_mandir}/man1/... ------------------------------------------------------- - While this is not a problem, I prefer to change the permission of files before %install section ends and not to use %attr when possible, i.e. ------------------------------------------------------- %install ....... chmod 0644 $RPM_BUILD_ROOT%{_mandir}/man1/* ------------------------------------------------------- However this is not a blocker.
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
------- Additional Comments From allisson@gmail.com 2007-10-19 12:27 EST ------- Update package:
Spec URL: http://allisson.fedorapeople.org/packages/tcputils/tcputils.spec SRPM URL: http://allisson.fedorapeople.org/packages/tcputils/tcputils-0.6.2-2.fc7.src....
Changes - Keep timestamps on man files
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-10-19 13:02 EST ------- Okay, approved.
------------------------------------------------------------ This package (tcputils) is APPROVED by me ------------------------------------------------------------
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
------- Additional Comments From allisson@gmail.com 2007-10-19 13:08 EST ------- New Package CVS Request ======================= Package Name: tcputils Short Description: Utilities for TCP programming in shell-scripts Owners: allisson Branches: FC-6 F-7 InitialCC: Cvsextras Commits: yes
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
allisson@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2007-10-21 13:13 EST ------- mass branching for F-8 has happened, so I assumed you also wanted a F-8 branch. cvs done.
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
allisson@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From allisson@gmail.com 2007-10-21 21:43 EST ------- All done, thank you for review.
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|devel |rawhide
allisson@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
------- Additional Comments From allisson@gmail.com 2008-06-10 08:01 EST ------- Package Change Request ====================== Package Name: tcputils New Branches: EL-4 EL-5
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: tcputils - Utilities for TCP programming in shell-scripts
https://bugzilla.redhat.com/show_bug.cgi?id=339631
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-06-10 12:07 EST ------- cvs done.
package-review@lists.fedoraproject.org