[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-10-05 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-05 12:44 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-10-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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 08:16 EST ---
Package Change Request
==
Package Name: corkscrew
New Branches: F-8

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-12 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-11 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-06-11 19:16 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-11 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||
   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-06-11 14:17 EST ---
(In reply to comment #18)
> I am unable to set the 'fedora-cvs' flag to '?'. It  gives an error message
> saying I am unauthorized to set the flag.

Perhaps fedora account system is on the way of syncing something.
You must be able to set the flag within a few hours(?), however
for now I set the flag.

Also, as now you are sponsored, FE-NEEDSPONSOR blocker is not needed.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-11 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-11 14:12 EST ---
I am unable to set the 'fedora-cvs' flag to '?'. It  gives an error message
saying I am unauthorized to set the flag.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-11 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-06-11 14:05 EST ---
New Package CVS Request
===
Package Name: corkscrew
Short Description: Tool for tunneling SSH through HTTP proxies.
Owners: [EMAIL PROTECTED]
Branches: FC-6, F-7
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-11 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-06-11 13:55 EST ---
I am now sponsoring

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-11 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-06-11 10:27 EST ---
Well,

* This package is okay.
* The package gengetopt is almost okay (for gengetopt, I have not
  checked in detail).

Okay.
-
  This package (corkscrew) is APPROVED by me
-

Please follow the procedure written on
http://fedoraproject.org/wiki/PackageMaintainers/Join
from "Get a Fedora Account"
When you requested someone to sponsor you (in the procedure
above), please make a note on this bug that you did so.

If you want to push this package also on F-7, you
also have to check:
http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT
after the URL above.

!! Well, recenctly Fedora package system changed a lot !!
   If you have some questions, please let me know.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-10 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-10 13:23 EST ---
I have removed the 'export CC=gcc' part, and here are the new files:

Here is the SPEC file:
https://fedoraproject.org/wiki/DebarshiRay?action=AttachFile&do=get&target=corkscrew-4.spec

Here is the SRPM file:
https://fedoraproject.org/wiki/DebarshiRay?action=AttachFile&do=get&target=corkscrew-2.0-4.fc7.src.rpm

I have also submitted a review request for GNU Gengetopt here:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=243607

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-08 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-08 14:20 EST ---
Well, I want to look at another review request you really
submitted (if any), or want to check your pre-review.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-08 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-08 14:08 EST ---
In case you are interested, I had made a compat-python24 package for Fedora
also. I did not submit it for package review since the main Python maintainer
opposed the idea of such a package. However the Zope package maintainer is
looking to use this for a Zope package for Fedora 7, which would most likely be
 put on a 3rd party repository.

Here is the SPEC file:
http://fedoraproject.org/wiki/DebarshiRay?action=AttachFile&do=get&target=compat-python.spec

Here is the SRPM file:
http://fedoraproject.org/wiki/DebarshiRay?action=AttachFile&do=get&target=compat-python-2.4.4-1.fc7.src.rpm

This almost a copy of the Python 2.4 package for FC6. I have just omitted out
some patches which cause build failures on Rawhide, plus a few renames to
prevent collision with the system's Python 2.5 files.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-08 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-08 13:45 EST ---
Acutally the line "export CC=gcc" was not needed.

Well, this package is okay, but as this is needsponsor ticket

-
NOTE: Before being sponsored:

This package will be accepted with another few work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to "show that you have an understanding 
of the process and of the packaging guidelines" as is described
on :
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a "pre-review" of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora package collection review requests which are waiting for someone to
review can be checked on:
http://fedoraproject.org/PackageReviewStatus/NEW.html
(NOTE: please don't choose "Merge Review")


Review guidelines are described mainly on:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-08 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-06-08 13:21 EST ---
Then assigning to me.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-08 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-08 13:09 EST ---
I promise that I'll assign the bug to myself and set fedora-review if I am doing
to do a full review and offer sponsorship.  I had intended to look at this
ticket again after my initial comments but never got around to it.  So many
tickets in the queue, you know.

Certainly if someone else wants to do the review and offer sponsorship, I'll be
quite happy to see that happen.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-08 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-08 12:37 EST ---
(In reply to comment #7)
> I don't know if "export CC=gcc" is needed
> and I don't have checked some details yet).

For what it is worth, I stole that "export CC=gcc" line from the Python
package's SPEC file. :-)



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-08 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-06-08 12:32 EST ---
Jason, are you reviewing this? For me the newest spec file
seems almost okay (though I don't know if "export CC=gcc" is needed
and I don't have checked some details yet).

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-06 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-06 14:37 EST ---
(In reply to comment #5)
> - please use one of the BuildRoot values listed at
>
http://fedoraproject.org/wiki/Packaging/Guidelines#head-b4fdd45fa76cbf54c885ef0836361319ab962473

Done.

> - please ditch the [ -d $RPM_BUILD_ROOT ] part in %install, it is not required
> any more

Done.

Here is the new and renamed SPEC file:
http://fedoraproject.org/wiki/DebarshiRay?action=AttachFile&do=get&target=corkscrew-3.spec

Here is the new and renamed SRPM:
http://fedoraproject.org/wiki/DebarshiRay?action=AttachFile&do=get&target=corkscrew-2.0-3.fc7.src.rpm

Please note that I have renamed the SPEC file just to avoid replacing the older 
one.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-06-06 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-06 09:43 EST ---
Just glanced over the package and there are a couple of issues:
- please use one of the BuildRoot values listed at
http://fedoraproject.org/wiki/Packaging/Guidelines#head-b4fdd45fa76cbf54c885ef0836361319ab962473
- please ditch the [ -d $RPM_BUILD_ROOT ] part in %install, it is not required
any more

The rest of the stuff seems fine. Please correct the above and I will do a
formal review. Note that I will not be allowed to approve the package because
you need a sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-05-19 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-19 23:59 EST ---
(In reply to comment #3)
> Please increment sa release field (and create a new src.rpm) each time you
> modify the spec file.

I thought that was only necessary after the package actually got into the
repositories. I was wrong.

Here is the new and renamed SPEC:
http://fedoraproject.org/wiki/DebarshiRay?action=AttachFile&do=get&target=corkscrew-2.spec

Here is the new and renamed SRPM:
http://fedoraproject.org/wiki/DebarshiRay?action=AttachFile&do=get&target=corkscrew-2.0-2.fc7.src.rpm

Please note that I have renamed the SPEC file just to avoid replacing the older 
one.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-05-19 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-19 05:47 EST ---
Please increment sa release field (and create a new src.rpm) each time you
modify the spec file. It makes easier for reviewers (and potential sponsors) to
track the evolution of the package. Of course, a change log entry should also be
added with each release bump.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-05-18 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-19 01:51 EST ---
(In reply to comment #1)
> The package builds fine; rpmlint says:
>   W: corkscrew summary-ended-with-dot Tool for tunneling SSH through HTTP 
> proxies.
> Should be a trivial fix.

Fixed.

> Note that Source0: should be a URL, so that spectool, for example, can 
> download
> the source.

Fixed.

> You must include the license text (COPYING) and really should include the rest
> of the included documentation (AUTHORS, ChangeLog, TODO) as %doc.

Done.

> Have you investigated how other distros package this program?  Debian 
> includes a
> manpage and a couple of bugfix patches.

I have imported the online manual page, and some modifications to README and
corkscrew.c from Debian. I have also put in a patch for a typo in README.


> Finally, are you sponsored? A quick search didn't turn you up as the owner of
> any other packages.  If you don't have a sponsor, you'll need to add
> FE-NEEDSPONSOR to the "Bug 239901 blocks" field and point out reviews that
> you've done or other packages you've submitted so that the sponsors will have
> enough information to decide whether they would like to sponsor you.

No. I am not sponsored and have not yet reviewed any packages for Fedora since I
am very new to packaging myself. My involvement with Fedora till now has been in
the form of bug reports and a successful Summer of Code proposal
(http://fedoraproject.org/wiki/SummerOfCode/2007/DebarshiRay). Apart from Fedora
I contribute to GNU too.

NB: The modified SRPM and SPEC file are available at the links mentioned in the
initial review request.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-05-15 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 239901] Review Request: corkscrew - Tool for tunneling SSH through HTTP proxies.

2007-05-15 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: corkscrew - Tool for tunneling SSH through HTTP 
proxies.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-05-15 17:59 EST ---
The package builds fine; rpmlint says:
  W: corkscrew summary-ended-with-dot Tool for tunneling SSH through HTTP 
proxies.
Should be a trivial fix.

Note that Source0: should be a URL, so that spectool, for example, can download
the source.

You must include the license text (COPYING) and really should include the rest
of the included documentation (AUTHORS, ChangeLog, TODO) as %doc.  INSTALL and
NEWS are generic and content-free so there's no need to include them.

Have you investigated how other distros package this program?  Debian includes a
manpage and a couple of bugfix patches.  When packaging software like this that
hasn't been touched in years, it's always worth looking at other distros to see
how they're handling bugfixes and such.

Finally, are you sponsored?  A quick search didn't turn you up as the owner of
any other packages.  If you don't have a sponsor, you'll need to add
FE-NEEDSPONSOR to the "Bug 239901 blocks" field and point out reviews that
you've done or other packages you've submitted so that the sponsors will have
enough information to decide whether they would like to sponsor you.

Here's a review:
* source files match upstream:
   0d0fcbb41cba4a81c4ab494459472086f377f9edb78a2e2238ed19b58956b0be  
   corkscrew-2.0.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
X license text included in source but not in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* debuginfo package looks complete.
X rpmlint has a valid complaint
* final provides and requires are sane.
O %check is not present; no test suite upstream.  Reviewer has no means to test 
  this software.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la droppings.
* not a GUI app.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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