Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mdk: new package #13325

Merged
merged 1 commit into from
Feb 22, 2016
Merged

mdk: new package #13325

merged 1 commit into from
Feb 22, 2016

Conversation

mjhoy
Copy link
Contributor

@mjhoy mjhoy commented Feb 21, 2016

this adds GNU's MDK suite.

a few notes:

  • I have declared stdenv.lib.platforms.all but have only tested on darwin 10.9 (and it builds and runs fine). I can test on linux tomorrow if you are interested.
  • I think many more dependencies may be added (guile, gtk, etc). This adds mixasm, mixvm binaries, and a few things to share (such as an emacs module).

thanks. also, this is my first pull request, apologies if this isn't how it is done.

@mjhoy
Copy link
Contributor Author

mjhoy commented Feb 21, 2016

just tested on nixos, works as well.

@fpletz fpletz added the 8.has: package (new) This PR adds a new package label Feb 22, 2016
@fpletz
Copy link
Member

fpletz commented Feb 22, 2016

Looks good. 👍

Did you test if the emacs stuff works? I don't use emacs, so I'm unable to test. :)

@mjhoy
Copy link
Contributor Author

mjhoy commented Feb 22, 2016

yep, (require mixal-mode) works.

@fpletz
Copy link
Member

fpletz commented Feb 22, 2016

Thanks!

fpletz added a commit that referenced this pull request Feb 22, 2016
@fpletz fpletz merged commit 230cf33 into NixOS:master Feb 22, 2016
@Janik-Haag Janik-Haag added the 12. first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: package (new) This PR adds a new package 12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants