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

add recipe for literate emacs starter kit #2227

Merged
merged 2 commits into from
Dec 5, 2014
Merged

Conversation

eschulte
Copy link
Contributor

No description provided.

@purcell
Copy link
Member

purcell commented Nov 30, 2014

Thanks!

You'll need either a literate-emacs-starter-kit.el or a literate-emacs-starter-kit-pkg.el -- probably the latter, in order to provide metadata. Though the "emacs" part of the name is redundant and should be removed.

In fact, I see you've actually got a Makefile which does the necessary generation, but MELPA doesn't run make or any other server-side scripts. Not sure where that leaves us here.

@eschulte
Copy link
Contributor Author

eschulte commented Dec 2, 2014

Ah, it's been too long since I touched ELPA stuff, and I tried to use a Makefile target used for GNU ELPA for MELPA. I just added a literate-starter-kit-pkg.el file to the starter kit repository, and updated my melpa branch pointed to by this pull request with a new recipe.

Please let me know if anything other changes are required.

Thanks,
Eric

@purcell
Copy link
Member

purcell commented Dec 4, 2014

Sorry for the delay getting back to this. Okay, the -pkg.el file looks good, but what else needs to go into the package? The recipe will need something like :files ("*.el" "*.org" "snippets") to include all the necessary files, I think.

  Including the names of directories which ideally should be created
  upon installation.
@eschulte
Copy link
Contributor Author

eschulte commented Dec 5, 2014

Done, thanks for catching that!

purcell added a commit that referenced this pull request Dec 5, 2014
add recipe for literate emacs starter kit
@purcell purcell merged commit 8128605 into melpa:master Dec 5, 2014
@purcell
Copy link
Member

purcell commented Dec 5, 2014

Merged, then, thanks! Is the idea that the user will need to run make after installation of the package, or does the startup code automate that?

@eschulte
Copy link
Contributor Author

eschulte commented Dec 6, 2014

Make is optional. Any .org files that are newer than the corresponding .el files are automatically tangled to .el files and then loaded by org-babel-load-file.

Thanks

@purcell
Copy link
Member

purcell commented Dec 6, 2014

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants