elisp-refs icon indicating copy to clipboard operation
elisp-refs copied to clipboard

Consider contributing this to Emacs?

Open phil-s opened this issue 8 years ago • 7 comments

Any tools directly related to elisp would be (especially) valuable to have as default functionality in Emacs.

Please do consider contributing this, while you still have only yourself to get permission from?

phil-s avatar Oct 01 '16 06:10 phil-s

This was mentioned on reddit too.

I'd be very willing to contribute it to Emacs: do you mean core, or GNU ELPA?

elisp-refs is heavily dependent on dash.el, but that's available on GNU ELPA. The other dependencies are not, but they're just conveniences.

I have a slight preference for giving it a month or two before trying to upstream it, just to get feedback etc. What do you think?

I'm also keen to keep development on GitHub, but that shouldn't be an issue (again, dash.el seems to cope).

Wilfred avatar Oct 01 '16 18:10 Wilfred

I was thinking of core, but I hadn't noticed the dependencies. They might well mean GNU ELPA is the practical place. It would certainly be nice if elisp programming tools were in core by default, though.

I believe you'll need to be careful about accepting pull requests. If you start merging code from people who have not signed copyright assignment papers, it's surely going to cause you problems down the track if you do want to get this included in core or GNU ELPA.

It might be interesting to find out how dash manages that issue.

phil-s avatar Oct 02 '16 09:10 phil-s

dash.el still takes patches from anyone, as I understand it: https://twitter.com/magnars/status/764741295994724352

That said, company and lispy both require copyright assignment. I know abo-abo has an account on the fencepost system, so it's easy for him to discover whether PR authors have assigned copyright.

Anyway, what's the process to get into GNU ELPA?

Wilfred avatar Oct 03 '16 01:10 Wilfred

OK, I've got permissions on GNU ELPA, so I should be able to add elisp-refs there. I'll need to add loop.el (only written by me, should be easy) and look at removing the other dependencies.

Wilfred avatar Nov 04 '16 18:11 Wilfred

OK, this should now be possible :)

Wilfred avatar Jan 11 '18 22:01 Wilfred

Hi!

Any chance for having this available on GNU ELPA? I think the easiest way to submit it is to make a post to [email protected] explaining what it does and that you want to submit it (attach a copy of elisp-refs.el if that is practical). From there, you usually get help by someone who knows exactly how this should be set up.

skangas avatar Oct 01 '21 01:10 skangas

Having recently added taxy to GNU ELPA, I can suggest checking the readme. You can submit a patch to the packages file that adds the package directly, and then other-Stefan will take it from there and ask you to make any further changes that might be needed.

alphapapa avatar Oct 01 '21 04:10 alphapapa