unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: Add libmikmod
@ 2013-10-22  0:00 David Thompson
  2013-10-23 19:19 ` Andreas Enge
  0 siblings, 1 reply; 6+ messages in thread
From: David Thompson @ 2013-10-22  0:00 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 199 bytes --]

Hello all,

I'm starting to work on packages for SDL and its extension libraries. 
They depend on some libraries that are not yet packaged. This is the 
first of these packages: libmikmod.

- Dave



[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-libmikmod.patch --]
[-- Type: text/x-patch; name="0001-gnu-Add-libmikmod.patch", Size: 3016 bytes --]

From 276a6fabc21c954b6cb3bc6ce707a86fc494d3e2 Mon Sep 17 00:00:00 2001
From: David Thompson <dthompson2@worcester.edu>
Date: Mon, 21 Oct 2013 19:50:01 -0400
Subject: [PATCH] gnu: Add libmikmod.

* gnu/packages/libmikmod.scm: New file.
* gnu-system.am (GNU_SYSTEM_MODULES): Add it.
---
 gnu-system.am              |  1 +
 gnu/packages/libmikmod.scm | 48 ++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 49 insertions(+)
 create mode 100644 gnu/packages/libmikmod.scm

diff --git a/gnu-system.am b/gnu-system.am
index 4ba1106..d281ee2 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -104,6 +104,7 @@ GNU_SYSTEM_MODULES =				\
   gnu/packages/libffi.scm			\
   gnu/packages/libidn.scm			\
   gnu/packages/libjpeg.scm			\
+  gnu/packages/libmikmod.scm			\
   gnu/packages/libphidget.scm			\
   gnu/packages/libpng.scm			\
   gnu/packages/libsigsegv.scm			\
diff --git a/gnu/packages/libmikmod.scm b/gnu/packages/libmikmod.scm
new file mode 100644
index 0000000..9b94155
--- /dev/null
+++ b/gnu/packages/libmikmod.scm
@@ -0,0 +1,48 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2013 David Thompson <dthompson2@worcester.edu>
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.
+
+(define-module (gnu packages libmikmod)
+  #:use-module (gnu packages)
+  #:use-module (guix licenses)
+  #:use-module (guix packages)
+  #:use-module (guix download)
+  #:use-module (guix build-system gnu)
+  #:use-module (gnu packages linux)
+  #:export (libmikmod))
+
+(define libmikmod
+  (package
+    (name "libmikmod")
+    (version "3.3.3")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append "mirror://sourceforge/mikmod/libmikmod/"
+                          version "/libmikmod-" version ".tar.gz"))
+             (sha256
+              (base32
+               "0dr4kgvhq9wf2riibh178c2al996spwwak6zffpv5n5bqmw29w3r"))))
+    (build-system gnu-build-system)
+    (inputs `(("alsa-lib" ,alsa-lib)))
+    (synopsis "Sound library that supports formats such as mod, s3m, it, and
+xm.")
+    (description
+     "MikMod is able to play a wide range of module formats, as well as
+digital sound files. It can take advantage of particular features of your
+system, such as sound redirection over the network.")
+    (license lgpl2.1)
+    (home-page "http://mikmod.sourceforge.net/")))
-- 
1.8.4.rc3


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] gnu: Add libmikmod
  2013-10-22  0:00 [PATCH] gnu: Add libmikmod David Thompson
@ 2013-10-23 19:19 ` Andreas Enge
  2013-10-23 22:14   ` David Thompson
  0 siblings, 1 reply; 6+ messages in thread
From: Andreas Enge @ 2013-10-23 19:19 UTC (permalink / raw)
  To: David Thompson; +Cc: guix-devel

Hello,

On Mon, Oct 21, 2013 at 08:00:50PM -0400, David Thompson wrote:
> I'm starting to work on packages for SDL and its extension
> libraries. They depend on some libraries that are not yet packaged.
> This is the first of these packages: libmikmod.

how about putting all of them into a module named sdl?

The configure phase outputs
   checking for X... no
Would it make sense to add xorg related inputs?

Andreas

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] gnu: Add libmikmod
  2013-10-23 19:19 ` Andreas Enge
@ 2013-10-23 22:14   ` David Thompson
  2013-10-25 20:39     ` Andreas Enge
  0 siblings, 1 reply; 6+ messages in thread
From: David Thompson @ 2013-10-23 22:14 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 680 bytes --]

On 10/23/2013 03:19 PM, Andreas Enge wrote:
> Hello,
>
> On Mon, Oct 21, 2013 at 08:00:50PM -0400, David Thompson wrote:
>> I'm starting to work on packages for SDL and its extension
>> libraries. They depend on some libraries that are not yet packaged.
>> This is the first of these packages: libmikmod.
> how about putting all of them into a module named sdl?

I will, but libmikmod is a library that is not directly associated with 
SDL, so I made a separate module.
>
> The configure phase outputs
>     checking for X... no
> Would it make sense to add xorg related inputs?
>
> Andreas
>
Yes, that would make sense. Updated patch attached.

Thanks for taking a look.

- Dave

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-libmikmod.patch --]
[-- Type: text/x-patch; name="0001-gnu-Add-libmikmod.patch", Size: 3086 bytes --]

From 859d90abe4ef96d8840f7bc218825629658e3220 Mon Sep 17 00:00:00 2001
From: David Thompson <dthompson2@worcester.edu>
Date: Mon, 21 Oct 2013 19:50:01 -0400
Subject: [PATCH] gnu: Add libmikmod.

* gnu/packages/libmikmod.scm: New file.
* gnu-system.am (GNU_SYSTEM_MODULES): Add it.
---
 gnu-system.am              |  1 +
 gnu/packages/libmikmod.scm | 50 ++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 51 insertions(+)
 create mode 100644 gnu/packages/libmikmod.scm

diff --git a/gnu-system.am b/gnu-system.am
index 4ba1106..d281ee2 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -104,6 +104,7 @@ GNU_SYSTEM_MODULES =				\
   gnu/packages/libffi.scm			\
   gnu/packages/libidn.scm			\
   gnu/packages/libjpeg.scm			\
+  gnu/packages/libmikmod.scm			\
   gnu/packages/libphidget.scm			\
   gnu/packages/libpng.scm			\
   gnu/packages/libsigsegv.scm			\
diff --git a/gnu/packages/libmikmod.scm b/gnu/packages/libmikmod.scm
new file mode 100644
index 0000000..a4e40bc
--- /dev/null
+++ b/gnu/packages/libmikmod.scm
@@ -0,0 +1,50 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2013 David Thompson <dthompson2@worcester.edu>
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.
+
+(define-module (gnu packages libmikmod)
+  #:use-module (gnu packages)
+  #:use-module (guix licenses)
+  #:use-module (guix packages)
+  #:use-module (guix download)
+  #:use-module (guix build-system gnu)
+  #:use-module (gnu packages linux)
+  #:use-module (gnu packages xorg)
+  #:export (libmikmod))
+
+(define libmikmod
+  (package
+    (name "libmikmod")
+    (version "3.3.3")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append "mirror://sourceforge/mikmod/libmikmod/"
+                          version "/libmikmod-" version ".tar.gz"))
+             (sha256
+              (base32
+               "0dr4kgvhq9wf2riibh178c2al996spwwak6zffpv5n5bqmw29w3r"))))
+    (build-system gnu-build-system)
+    (inputs `(("alsa-lib" ,alsa-lib)
+              ("libx11" ,libx11)))
+    (synopsis "Sound library that supports formats such as mod, s3m, it, and
+xm.")
+    (description
+     "MikMod is able to play a wide range of module formats, as well as
+digital sound files. It can take advantage of particular features of your
+system, such as sound redirection over the network.")
+    (license lgpl2.1)
+    (home-page "http://mikmod.sourceforge.net/")))
-- 
1.8.4.rc3


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] gnu: Add libmikmod
  2013-10-23 22:14   ` David Thompson
@ 2013-10-25 20:39     ` Andreas Enge
  2013-10-25 23:49       ` David Thompson
  0 siblings, 1 reply; 6+ messages in thread
From: Andreas Enge @ 2013-10-25 20:39 UTC (permalink / raw)
  To: David Thompson; +Cc: guix-devel

On Wed, Oct 23, 2013 at 06:14:17PM -0400, David Thompson wrote:
> I will, but libmikmod is a library that is not directly associated
> with SDL, so I made a separate module.

Well, I am a bit wary about creating lots of files for small packages.
If it occurs essentially as a dependency of sdl, I would be happy to
package it inside the sdl module. But we do not have a coherent policy
on packages vs. modules.

Concerning style:
In 'uri', the argument 'version' to 'string-append' should be aligned
with the first argument of the line above, that is, "mirror...".
I am not sure what to do about the synopsis spanning two lines; I think
we do not have a strict 80 column policy, so it could be put into one
longer line.
After copyright, I have '??' instead of '©'; maybe the file is not
in utf-8 format?

In any case, it compiles well, and after addressing the small stylistic
comments, I support pushing it.

Andreas

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] gnu: Add libmikmod
  2013-10-25 20:39     ` Andreas Enge
@ 2013-10-25 23:49       ` David Thompson
  2013-10-26 12:18         ` Andreas Enge
  0 siblings, 1 reply; 6+ messages in thread
From: David Thompson @ 2013-10-25 23:49 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 1342 bytes --]

On 10/25/2013 04:39 PM, Andreas Enge wrote:
> On Wed, Oct 23, 2013 at 06:14:17PM -0400, David Thompson wrote:
>> I will, but libmikmod is a library that is not directly associated
>> with SDL, so I made a separate module.
>
> Well, I am a bit wary about creating lots of files for small packages.
> If it occurs essentially as a dependency of sdl, I would be happy to
> package it inside the sdl module. But we do not have a coherent policy
> on packages vs. modules.

Okay, that sounds reasonable. I have renamed the module.

>
> Concerning style:
> In 'uri', the argument 'version' to 'string-append' should be aligned
> with the first argument of the line above, that is, "mirror...".

Silly mistake. Fixed.

> I am not sure what to do about the synopsis spanning two lines; I think
> we do not have a strict 80 column policy, so it could be put into one
> longer line.

Shortened the synopsis.

> After copyright, I have '??' instead of '©'; maybe the file is not
> in utf-8 format?

Mark Weaver pointed out that my attachment has no MIME type specifying 
it to be UTF-8 text. I have tweaked an Icedove setting, so hopefully 
things will be okay this time around. The patch *should* be inlined.

>
> In any case, it compiles well, and after addressing the small stylistic
> comments, I support pushing it.
>
> Andreas
>

Thanks.

- Dave


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-libmikmod.patch --]
[-- Type: text/x-patch; name="0001-gnu-Add-libmikmod.patch", Size: 3005 bytes --]

From a209fd040de0f026ddcd38d0c644517769b0802a Mon Sep 17 00:00:00 2001
From: David Thompson <dthompson2@worcester.edu>
Date: Mon, 21 Oct 2013 19:50:01 -0400
Subject: [PATCH] gnu: Add libmikmod.

* gnu/packages/sdl.scm: New file.
* gnu-system.am (GNU_SYSTEM_MODULES): Add it.
---
 gnu-system.am        |  1 +
 gnu/packages/sdl.scm | 49 +++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 50 insertions(+)
 create mode 100644 gnu/packages/sdl.scm

diff --git a/gnu-system.am b/gnu-system.am
index 4ba1106..1c24607 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -159,6 +159,7 @@ GNU_SYSTEM_MODULES =				\
   gnu/packages/samba.scm			\
   gnu/packages/scheme.scm			\
   gnu/packages/screen.scm			\
+  gnu/packages/sdl.scm				\
   gnu/packages/shishi.scm			\
   gnu/packages/skribilo.scm			\
   gnu/packages/smalltalk.scm			\
diff --git a/gnu/packages/sdl.scm b/gnu/packages/sdl.scm
new file mode 100644
index 0000000..ceb2173
--- /dev/null
+++ b/gnu/packages/sdl.scm
@@ -0,0 +1,49 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2013 David Thompson <dthompson2@worcester.edu>
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.
+
+(define-module (gnu packages sdl)
+  #:use-module (gnu packages)
+  #:use-module (guix licenses)
+  #:use-module (guix packages)
+  #:use-module (guix download)
+  #:use-module (guix build-system gnu)
+  #:use-module (gnu packages linux)
+  #:use-module (gnu packages xorg)
+  #:export (libmikmod))
+
+(define libmikmod
+  (package
+    (name "libmikmod")
+    (version "3.3.3")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append "mirror://sourceforge/mikmod/libmikmod/"
+                                 version "/libmikmod-" version ".tar.gz"))
+             (sha256
+              (base32
+               "0dr4kgvhq9wf2riibh178c2al996spwwak6zffpv5n5bqmw29w3r"))))
+    (build-system gnu-build-system)
+    (inputs `(("alsa-lib" ,alsa-lib)
+              ("libx11" ,libx11)))
+    (synopsis "Library for module sound formats.")
+    (description
+     "MikMod is able to play a wide range of module formats, as well as
+digital sound files. It can take advantage of particular features of your
+system, such as sound redirection over the network.")
+    (license lgpl2.1)
+    (home-page "http://mikmod.sourceforge.net/")))
-- 
1.8.4.rc3


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] gnu: Add libmikmod
  2013-10-25 23:49       ` David Thompson
@ 2013-10-26 12:18         ` Andreas Enge
  0 siblings, 0 replies; 6+ messages in thread
From: Andreas Enge @ 2013-10-26 12:18 UTC (permalink / raw)
  To: David Thompson; +Cc: guix-devel

On Fri, Oct 25, 2013 at 07:49:12PM -0400, David Thompson wrote:
> >After copyright, I have '??' instead of '©'; maybe the file is not
> >in utf-8 format?
> Mark Weaver pointed out that my attachment has no MIME type
> specifying it to be UTF-8 text. I have tweaked an Icedove setting,
> so hopefully things will be okay this time around. The patch
> *should* be inlined.

Well, the copyright symbol still did not print; but this was easy to fix,
and if you continue work on the sdl module, there will be no need to add
copyright symbols for the moment.

I just pushed the patch, thanks for your contribution!

Andreas

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2013-10-26 12:18 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-22  0:00 [PATCH] gnu: Add libmikmod David Thompson
2013-10-23 19:19 ` Andreas Enge
2013-10-23 22:14   ` David Thompson
2013-10-25 20:39     ` Andreas Enge
2013-10-25 23:49       ` David Thompson
2013-10-26 12:18         ` Andreas Enge

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).