Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 30 Dec 2008 23:55:08 GMT
From:      Michael W Lucas <mwlucas@blackhelicopters.org>
To:        freebsd-gnats-submit@FreeBSD.org
Subject:   www/130070: 8.0-CURRENT
Message-ID:  <200812302355.mBUNt8fj032747@www.freebsd.org>
Resent-Message-ID: <200812310000.mBV00Ilw069823@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help

>Number:         130070
>Category:       www
>Synopsis:       8.0-CURRENT
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-www
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Wed Dec 31 00:00:18 UTC 2008
>Closed-Date:
>Last-Modified:
>Originator:     Michael W Lucas
>Release:        8.0-CURRENT
>Organization:
none
>Environment:
FreeBSD stretchlimo.blackhelicopters.org 8.0-CURRENT FreeBSD 8.0-CURRENT #16: Tue Dec 16 10:22:25 EST 2008     mwlucas@stretchlimo.blackhelicopters.org:/usr/obj/usr/src/sys/GENERIC  i386
>Description:
I just submitted a ports PR with a patch attached.  The Web interface offers the following instructions for uploading a patch.

And/or patch file (100KB max, .txt extension recommended): 

The word "recommendation" implies that it's not strictly required.  I ignored the recommendation, and listed my patch as it existed.  (I knew durn well that the committer would have to rename it to the name I gave it, anyway.)

The submission was rejected because the patch did not have the extension .txt.  I do not object to this behavior, but I would suggest that the word "recommended" be changed to "required."

Of course, I stumbled over this less than a month after (willingly) relenquishing my doc commit bit...  figures.
>How-To-Repeat:
Go to http://www.freebsd.org/send-pr.html.  Try to submit a PR with a patch with a name such as 

patch-bin_flow-rptfmt

While this (or something much like it) is how the file name will appear when committed, send-pr.html rejected it.

Had it said "required," I would have changed the file name before even trying to submit.

>Fix:
Change "recommended" to "required," perhaps listing any other types that the script will take.

>Release-Note:
>Audit-Trail:
>Unformatted:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200812302355.mBUNt8fj032747>