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

The first commit from #44 #47

Merged
merged 1 commit into from Oct 21, 2015
Merged

The first commit from #44 #47

merged 1 commit into from Oct 21, 2015

Conversation

tarsius
Copy link
Member

@tarsius tarsius commented Oct 21, 2015

In #44 I said you should merge only the first commit, but you merged both. I then pointed out that you should revert the second commit and you reverted both. This new pull request allows you to merge the first commit again - and nothing else.

`cl' is required again for the sake of `lexical-let', but at least for
recent Emacsen only at compile time.  At runtime `cl' is only required
right before the use of `lexical-let', which is on a code path only
used by older Emacsen.

Also add the `cl-' prefix to some symbols.

This completes #43.
@m2ym
Copy link
Contributor

m2ym commented Oct 21, 2015

Thanks. I supposed to cherry-pick the first commit very soon.

m2ym added a commit that referenced this pull request Oct 21, 2015
@m2ym m2ym merged commit 75c73f7 into emacsorphanage:master Oct 21, 2015
@tarsius
Copy link
Member Author

tarsius commented Oct 21, 2015

I figured you were a bit busy, or something :-)

@tarsius tarsius deleted the cl-lib branch October 21, 2015 15:40
@m2ym
Copy link
Contributor

m2ym commented Oct 21, 2015

Sorry. BTW, are you interested in to become a new maintainer of direx? As you said, I don't have much time to work on this project. I can transfer the repo.

@tarsius
Copy link
Member Author

tarsius commented Oct 22, 2015

No, sorry. I also don't have enough time.

@m2ym
Copy link
Contributor

m2ym commented Oct 23, 2015

OK. Thanks, anyway.

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

Successfully merging this pull request may close these issues.

None yet

2 participants