[Bug 197747] ports-mgmt/portlint : request to warn on patches that don't have UTC in "---" line

bugzilla-noreply at freebsd.org bugzilla-noreply at freebsd.org
Mon Feb 16 21:13:42 UTC 2015


https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=197747

            Bug ID: 197747
           Summary: ports-mgmt/portlint : request to warn on patches that
                    don't have UTC in "---" line
           Product: Ports & Packages
           Version: Latest
          Hardware: Any
                OS: Any
            Status: New
          Severity: Affects Only Me
          Priority: ---
         Component: Individual Port(s)
          Assignee: marcus at FreeBSD.org
          Reporter: marino at FreeBSD.org
          Assignee: marcus at FreeBSD.org
             Flags: maintainer-feedback?(marcus at FreeBSD.org)

It is highly desirable that patches are created with "make makepatch" which
results in headers like this:

--- lib/krb5/ccache/cc_file.c.orig     2014-10-15 23:55:10 UTC
+++ lib/krb5/ccache/cc_file.c
@@ -136,6 +136,7 @@ get_size(krb5_context context, krb5_ccac


Unfortunately, a lot of people still use diff, even reverting the "good" format
above, resulting in something like this:

--- lib/krb5/ccache/cc_file.c.orig     2014-10-15 16:55:10.000000000 -0700
+++ lib/krb5/ccache/cc_file.c  2015-02-12 13:00:31.967166267 -0800
@@ -136,6 +136,7 @@

Notice the desired "-p" option is lost as well.



I think it would be a good idea to check the "--- " line of each patch to make
sure it ends in "UTC" and give an appropriate warning and recommendation to use
"make makepatch" instead.  Since there can be more than one "--- " line per
patch, each matching line should be checked IMO.

--- Comment #1 from Bugzilla Automation <bugzilla at FreeBSD.org> ---
Auto-assigned to maintainer marcus at FreeBSD.org

-- 
You are receiving this mail because:
You are the assignee for the bug.


More information about the freebsd-ports-bugs mailing list