Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 01 Jul 2012 19:44:38 +0200
From:      Bernhard =?ISO-8859-1?Q?Fr=F6hlich?= <decke@bluelife.at>
To:        Fernando =?ISO-8859-1?Q?Apestegu=ED?= <fernando.apesteguia@gmail.com>, freebsd-ports@freebsd.org
Subject:   Re: [new-port] JDownloader
Message-ID:  <1341164678.10569.2.camel@Nokia-N900-42-11>
In-Reply-To: <CAGwOe2bbBCOqcTpudF8sGKNQYf46a8xRdBKTP12OwQ6hesOTmw@mail.gmail.com>
References:  <CAGwOe2bbBCOqcTpudF8sGKNQYf46a8xRdBKTP12OwQ6hesOTmw@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On So.,   1. Jul. 2012 19:32:58 CEST, Fernando Apesteguía <fernando.apesteguia@gmail.com> wrote:

> Hi all,
> 
> I have a preliminary version of a port for JDownloader[1] since it
> seems to be a wanted port[2]
> 
> Here is the Makefile:
> 
> -----------------------------------------------------------------------
> # New ports collection makefile for:    jdownloader
> # Date created:                                       Sun Jul 1 2012
> # Whom:                    Fernando Apesteguia <fernando.apesteguia@gmail.com>
> #
> # $FreeBSD$
> 
> PORTNAME=    jdownloader
> PORTVERSION=    1.0
> CATEGORIES=    java ftp
> MASTER_SITES=    http://212.117.163.148/
> DISTNAME=    jd.sh
> EXTRACT_SUFX=
> 
> MAINTAINER=    fernando.apesteguia@gmail.com
> COMMENT=    Download management for one-click-hosters
> 
> RUN_DEPENDS=    ${LOCALBASE}/bin/java:${PORTSDIR}/java/diablo-jdk16
> RUN_DEPENDS+=    ${LOCALBASE}/bin/wget:${PORTSDIR}/ftp/wget
> 
> USE_JAVA=    yes
> JAVA_VERSION=    1.6+
> NO_BUILD=    yes
> 
> WRKSRC=        ${WRKDIR}
> 
> do-extract:
>     @${MKDIR} ${WRKDIR}
>     @${CP} ${DISTDIR}/jd.sh ${WRKSRC}
> 
> do-install:
>     @${MKDIR} ${PREFIX}/${PORTNAME}
>     @${INSTALL_SCRIPT} ${WRKSRC}/jd.sh ${PREFIX}/bin
> 
> post-install:
>     @${ECHO_MSG} ""
>     @${ECHO_MSG}
> "*************************************************************************"
>     @${ECHO_MSG} JDownloader will update itself to the latest version when
> it runs for the    @${ECHO_MSG} first time. Please run
> ${LOCALBASE}/bin/jd.sh    @${ECHO_MSG} to complete the installation
>     @${ECHO_MSG}
> "*************************************************************************"
>     @${ECHO_MSG} ""    @${CAT} ${PKGMESSAGE}
> 
> .include <bsd.port.mk>
> -----------------------------------------------------------------------
> 
> The port just needs a simple patch file (patch-jd.sh):
> 
> -----------------------------------------------------------------------
> --- jd.sh.orig    2012-07-01 14:23:07.000000000 +0200
> +++ jd.sh    2012-07-01 14:24:15.000000000 +0200
> @@ -1,4 +1,4 @@
> -#!/bin/bash
> +#!/bin/sh
>   #JD Installer/Starter Version 0.2
>   #by Jiaz(JD-Team), jiaz@jdownloader.org
>   #You need at least:
> -----------------------------------------------------------------------
> 
> The jd.sh runs fine with sh(1) so there is no need for a new run
> dependency for bash(1).
> 
> The port installs one single file (jd.sh). This script when executed,
> checks for the presence of the JDownloader jar files under the ~/.jd
> directory. If they don't exist it starts the uploader application and
> downloads the latest files. If the files were already present, it
> launches the installation and if the installation is already present,
> it launches the program.
> Uninstalling the port uninstalls the jd.sh file but _not_ the actual
> application that might be installed by the users in their respective
> home directories. Taking into account that JDownloader can be
> installed anyway just by downloading the source I don't suppose this
> is a concern. Since the program can act as a server (it installs a
> plugin for remote controlling the application similarly to what
> transmission torrent does), it shows a message after the installation.
> 
> I haven't submitted a PR yet, so any suggestions are welcomed.
> 
> Thanks in advance.
> 
> [1] http://jdownloader.org/
> [2] http://wiki.freebsd.org/WantedPorts

Is there a reason why you explicitly add diablo-jdk16 as runtime dependency? The use_java stuff should be sufficient.

Why have you decided to download and install the jar files on first run of the shellscript? That doesn't sound like a good idea to me because it breaks proper updates, deinstall and creates useless binary packages.



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