"Dmitrij D. Czarkoff" <[email protected]> writes:
> LEVAI Daniel said:
>> I don't mind having the pwsafe convert utility around, I just tought
>> that generally the expectation is that if something gets installed, then
>> it should (or supposed to) work :) If it's enough (from the user point
>> of view) that perl will warn the user that some modules are necessary
>> for running said program, then I'm okay with installing it henceforward.
>> If this is the case, I guess the pkg-readme could also stay, but with
>> some modifications that mention the perl module dependency. Is this an
>> acceptable standard in the ports tree?
>
> I would drop the readme from the patch and either drop it from PLIST or
> (if it is worth having) make a subpackage.
I don't think the complexity of subpackages are worth it, I'd just ship
the script as is, with a note in the readme.
Index: Makefile
===================================================================
RCS file: /cvs/ports/security/kc/Makefile,v
retrieving revision 1.7
diff -u -p -r1.7 Makefile
--- Makefile 7 Aug 2013 21:32:34 -0000 1.7
+++ Makefile 23 Dec 2013 17:07:14 -0000
@@ -2,12 +2,15 @@
COMMENT = console based password storing application
-DISTNAME = kc-2.3.0
-REVISION = 0
-CATEGORIES = security
+GH_VER = 2.3.2
+PROJECT = kc
+DISTNAME = ${PROJECT}-${GH_VER}
+CATEGORIES = security
-HOMEPAGE = https://code.google.com/p/keychain
-MASTER_SITES = https://keychain.googlecode.com/files/
+HOMEPAGE = http://github.com/levaidaniel/${PROJECT}
+MASTER_SITES = https://github.com/levaidaniel/${PROJECT}/archive/
+
+DISTFILES = ${DISTNAME}{${GH_VER}}${EXTRACT_SUFX}
MAINTAINER = LEVAI Daniel <[email protected]>
@@ -19,13 +22,7 @@ WANTLIB = c crypto edit ncursesw xml2 pc
LIB_DEPENDS = textproc/libxml \
devel/pcre
-# only required for the pwsafe_to_kc.pl utility,
-# and not for kc itself.
-RUN_DEPENDS = www/p5-HTML-Parser
-
MAKE_ENV += HAVE_PCRE=yes
-
-TEST_TARGET =test
post-install:
${INSTALL_DATA_DIR} ${PREFIX}/share/kc
Index: distinfo
===================================================================
RCS file: /cvs/ports/security/kc/distinfo,v
retrieving revision 1.4
diff -u -p -r1.4 distinfo
--- distinfo 21 Mar 2013 09:48:45 -0000 1.4
+++ distinfo 21 Dec 2013 22:03:27 -0000
@@ -1,2 +1,2 @@
-SHA256 (kc-2.3.0.tar.gz) = SNp9q8TaMX7ACQLN2BGwsEAIkjoYsvKiGUjAQQNWsq4=
-SIZE (kc-2.3.0.tar.gz) = 43625
+SHA256 (kc-2.3.2.tar.gz) = 1Ap74wB55M/EI+GqK127Dm8LNInx0OCQzWZyi8Mo/Ng=
+SIZE (kc-2.3.2.tar.gz) = 44779
Index: pkg/README
===================================================================
RCS file: /cvs/ports/security/kc/pkg/README,v
retrieving revision 1.1
diff -u -p -r1.1 README
--- pkg/README 25 Jan 2013 11:09:20 -0000 1.1
+++ pkg/README 23 Dec 2013 17:18:52 -0000
@@ -4,6 +4,7 @@ $OpenBSD: README,v 1.1 2013/01/25 11:09:
| Running ${FULLPKGNAME} on OpenBSD
+-----------------------------------------------------------------------
-kc comes with a tiny and simple utility which can convert an exported pwsafe
-database to a kc compatible XML database (that can be imported to it later).
-It is installed under ${PREFIX}/share/kc/
+kc comes with a tiny and simple utility which can convert an exported
+pwsafe database to a kc compatible XML database (that can be imported to
+it later). It is installed under ${PREFIX}/share/kc/. Note that this
+script requires Perl modules provided by the p5-HTML-Parser package.
--
jca | PGP: 0x06A11494 / 61DB D9A0 00A4 67CF 2A90 8961 6191 8FBF 06A1 1494
