From: Alex Kost <alezost@gmail.com>
To: Alex Vong <alexvong1995@gmail.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] gnu: ladspa: Update urls.
Date: Thu, 12 Nov 2015 21:11:34 +0300 [thread overview]
Message-ID: <87si4bqe95.fsf@gmail.com> (raw)
In-Reply-To: <CADrxHD966OU_cFRVPDpos3Pdtg_wZfv14nDS0k4AktL_BTvj8w@mail.gmail.com> (Alex Vong's message of "Thu, 12 Nov 2015 22:34:22 +0800")
Alex Vong (2015-11-12 17:34 +0300) wrote:
[...]
> diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
> index a6899a0..9ed21bd 100644
> --- a/gnu/packages/audio.scm
> +++ b/gnu/packages/audio.scm
> @@ -758,7 +758,9 @@ plugin function as a JACK application.")
> (source (origin
> (method url-fetch)
> (uri (string-append
> - "http://www.ladspa.org/download/ladspa_sdk_"
> + "http://slackware.org.uk/slacky/slackware-13.0/multimedia/ladspa/"
> + version
> + "/src/ladspa_sdk_"
> version
> ".tgz"))
Please keep lines within 78 columns. Split that long string, for
example like this:
(uri (string-append
"http://slackware.org.uk/slacky/slackware-13.0"
"/multimedia/ladspa/" version
"/src/ladspa_sdk_" version ".tgz"))
> (sha256
> @@ -781,7 +783,8 @@ plugin function as a JACK application.")
> (("^CC.*") "CC = gcc\n")
> (("^CPP.*") "CPP = g++\n"))))
> (alist-delete 'build %standard-phases))))
> - (home-page "http://ladspa.org")
> + (home-page "https://community.ardour.org/files/doxygen/ladspa_8h_source.html") ; Since the home page is gone,
> + ;; we provide a link to the self-documented header file.
This ^^^ is an unusual way to write a comment. I think it's better to
do it like this:
;; Since the home page is gone, we provide a link to the
;; self-documented header file.
(home-page
"https://community.ardour.org/files/doxygen/ladspa_8h_source.html")
Also as the home-page line is too long, I split it into 2 lines.
--
Alex
next prev parent reply other threads:[~2015-11-12 18:11 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-12 13:51 [PATCH] gnu: ladspa: Update urls Alex Vong
2015-11-12 14:27 ` Efraim Flashner
2015-11-12 14:34 ` Alex Vong
2015-11-12 18:11 ` Alex Kost [this message]
2015-11-12 16:26 ` Ludovic Courtès
-- strict thread matches above, loose matches on Subject: below --
2015-11-12 18:36 Alex Vong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87si4bqe95.fsf@gmail.com \
--to=alezost@gmail.com \
--cc=alexvong1995@gmail.com \
--cc=guix-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.