remove bundled dropdown-list.el #370

Merged
merged 1 commit into from May 2, 2013

Conversation

Projects
None yet
2 participants
Contributor

tarsius commented Mar 12, 2013

It is available from melpa, so no need to bundle it.

remove bundled dropdown-list.el
It is available from melpa, so no need to bundle it.
Contributor

tarsius commented Mar 12, 2013

fixes #356

joaotavora added a commit that referenced this pull request May 2, 2013

Merge pull request #370 from tarsius/master
remove bundled dropdown-list.el

@joaotavora joaotavora merged commit 7172a45 into joaotavora:master May 2, 2013

Owner

joaotavora commented May 2, 2013

Thanks! And sorry for the gigantic delay in analysing this.

Contributor

tarsius commented May 2, 2013

Np. I have seen much longer delays :-|

I have just created a pull request to update the melpa recipe: milkypostman/melpa#740

Since dropdown-list is only an optional dependency you don't have to add it to Package-Requires (and I think you shouldn't). But you should update the library commentary as it still sais:

;;   The `dropdown-list.el' extension is bundled with YASnippet,

@tarsius tarsius referenced this pull request in melpa/melpa May 2, 2013

Merged

yasnippet no longer bundles dropdown-list #740

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment