Bug 221818 - [PATCH] lang/clojure-mode.el: Maintainer update
Summary: [PATCH] lang/clojure-mode.el: Maintainer update
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Tobias Kortkamp
URL: http://portsmon.freebsd.org/portoverv...
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-08-25 20:39 UTC by SPPH
Modified: 2017-11-26 20:48 UTC (History)
2 users (show)

See Also:


Attachments
[PATCH] Makefile maintainer patch for clojure-mode.el (297 bytes, patch)
2017-08-25 20:39 UTC, SPPH
no flags Details | Diff
Patch moving clojure-mode.el-2.2.1 to clojure-mode.el-5.6.1 (137.13 KB, patch)
2017-08-27 18:47 UTC, SPPH
no flags Details | Diff
Patch files for moving clojure-mode.el-2.2.1 to clojure-mode.el-5.6.1 (137.12 KB, patch)
2017-08-31 19:39 UTC, SPPH
no flags Details | Diff
updated Makefile which fetches latest version of Clojure-mode (1.02 KB, patch)
2017-09-02 17:47 UTC, SPPH
no flags Details | Diff
Updated pkg-plist to reflect new files included in newest versions of Clojure-mode (326 bytes, patch)
2017-09-02 17:48 UTC, SPPH
no flags Details | Diff
Updated distinfo of Clojure-mode (505 bytes, patch)
2017-09-02 17:48 UTC, SPPH
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description SPPH 2017-08-25 20:39:54 UTC
Created attachment 185766 [details]
[PATCH] Makefile maintainer patch for clojure-mode.el

Hi, I'd like to become the maintainer for the lang/Clojure-mode.el port.
Comment 1 Tobias Kortkamp freebsd_committer freebsd_triage 2017-08-27 09:29:32 UTC
(In reply to SPPH from comment #0)
Sure, do you have a patch that brings the port to a newer version?
Comment 2 SPPH 2017-08-27 18:47:01 UTC
Created attachment 185822 [details]
Patch moving clojure-mode.el-2.2.1 to clojure-mode.el-5.6.1

Please find attached patch file for moving from clojure-mode.el-2.2.1 to clojure-mode.el-5.6.1.

Confirmed working amd64 FreeBSD version 11.1-release
Comment 3 Tobias Kortkamp freebsd_committer freebsd_triage 2017-08-29 15:29:37 UTC
Comment on attachment 185822 [details]
Patch moving clojure-mode.el-2.2.1 to clojure-mode.el-5.6.1

I do not know what to do with this?

I do mean a patch for actually updating the _port_ to 5.6.1.
Comment 4 SPPH 2017-08-31 19:39:19 UTC
Created attachment 185957 [details]
Patch files for moving clojure-mode.el-2.2.1 to clojure-mode.el-5.6.1

Hi sorry for the confusion.

I hope this helps:

I've attached one file:

There is actually only two files that need to be changed between the versions of clojure-mode-2.1.1 and clojure-mode-5.6.1, clojure-mode.el and the test.clj files; patch-clojure-mode.el.5.6.1.diff is the patch file for the entire project compared to the previous clojure-mode-2.2.1 file.

I do apologize for any confusion, this is my first time patching something for the FreeBSD community; if this isn't right I'll just use the svn way.
Comment 5 Tobias Kortkamp freebsd_committer freebsd_triage 2017-08-31 21:21:33 UTC
(In reply to SPPH from comment #4)
> I've attached one file:
It's the same thing again.

> There is actually only two files that need to be changed between the
> versions of clojure-mode-2.1.1 and clojure-mode-5.6.1,
>  clojure-mode.el and the test.clj files;
> patch-clojure-mode.el.5.6.1.diff is the patch file for the entire
> project compared to the previous clojure-mode-2.2.1 file.
Hmm, I think you have misunderstood me and are looking at this from
the wrong level.  Generally ports fetch source tarballs provided by
upstream projects and do not include the entire upstream projects's
changeset for going from version A to B.  (The ports tree would be
gigantic if we'd do this for all ports.)

You can start by bumping PORTVERSION in the port's Makefile and go
from there.  I'd suggest studying the Porter's Handbook [1] a bit more
however.  If you need more help or guidance I'd suggest you open a
thread on the FreeBSD Forums [2] or send an email to the
ports@FreeBSD.org mailing list.

[1] https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/index.html
[2] https://forums.freebsd.org
Comment 6 SPPH 2017-09-02 17:47:29 UTC
Created attachment 185997 [details]
updated Makefile which fetches latest version of Clojure-mode

Updated the clojure-mode.el/ Makefile to fetch the latest clojure-mode from the github repository.
Comment 7 SPPH 2017-09-02 17:48:22 UTC
Created attachment 185998 [details]
Updated pkg-plist to reflect new files included in newest versions of Clojure-mode

Updated pkg-plist to reflect new files included in newest versions of Clojure-mode.
Comment 8 SPPH 2017-09-02 17:48:57 UTC
Created attachment 185999 [details]
Updated distinfo of Clojure-mode

Updated distinfo of Clojure-mode
Comment 9 commit-hook freebsd_committer freebsd_triage 2017-09-03 11:12:33 UTC
A commit references this bug:

Author: tobik
Date: Sun Sep  3 11:11:58 UTC 2017
New revision: 449199
URL: https://svnweb.freebsd.org/changeset/ports/449199

Log:
  lang/clojure-mode.el: Update to 5.6.1

  PR:		221818
  Submitted by:	harrigan-s@outlook.com (new maintainer)

Changes:
  head/lang/clojure-mode.el/Makefile
  head/lang/clojure-mode.el/distinfo
  head/lang/clojure-mode.el/pkg-message
  head/lang/clojure-mode.el/pkg-plist
Comment 10 Tobias Kortkamp freebsd_committer freebsd_triage 2017-09-03 11:13:28 UTC
Committed with some changes. Thanks!

+PORTREVISION=	5

When PORTVERSION is bumped, then PORTREVISION should be reset i.e. it
needs to be removed.  PORTREVISION is only bumped when we do packaging
changes to the port (e.g. installing files in a different location,
adding or removing dependencies, etc.) without moving to a newer
version.  For 2.1.1 we have done this 2 times (=> PORTREVISION=2)
after the update to 2.1.1.

+EXTRACT_SUFX=	.tar.gz
+GH_TAGNAME=	${PORTVERSION}

These are set to the default values, so there is no reason to specify
them in the port yourself.

Generally it'd be easier for us if submit port changes as one patch
instead of several individual patches.
Comment 11 vali gholami 2017-11-26 20:48:21 UTC
MARKED AS SPAM