er #83

Open
yrashk opened this Issue Feb 7, 2011 · 10 comments

Comments

Projects
None yet
2 participants
@nox

This comment has been minimized.

Show comment Hide comment
@nox

nox Feb 12, 2011

Contributor

We should, seems like it's more erlang-friendly than erldis. Should we package erldis too?

Contributor

nox commented Feb 12, 2011

We should, seems like it's more erlang-friendly than erldis. Should we package erldis too?

@yrashk

This comment has been minimized.

Show comment Hide comment
@yrashk

yrashk Feb 13, 2011

Owner

I think erldis is already packaged:

$ agner spec erldis
{name,"erldis"}.
{authors,["iValentino Volonghi","Jacob Perkins","Eric Cestari","tonyg"]}.
{description,"redis erlang client library"}.
{homepage,"https://github.com/cstar/erldis"}.
{rebar_compatible,true}.
{license,"MIT","LICENSE"}.
{url,{git,"https://github.com/cstar/erldis.git",{branch,"master"}}}.
{applications,[erldis]}.
Owner

yrashk commented Feb 13, 2011

I think erldis is already packaged:

$ agner spec erldis
{name,"erldis"}.
{authors,["iValentino Volonghi","Jacob Perkins","Eric Cestari","tonyg"]}.
{description,"redis erlang client library"}.
{homepage,"https://github.com/cstar/erldis"}.
{rebar_compatible,true}.
{license,"MIT","LICENSE"}.
{url,{git,"https://github.com/cstar/erldis.git",{branch,"master"}}}.
{applications,[erldis]}.
@nox

This comment has been minimized.

Show comment Hide comment
@nox

nox Feb 13, 2011

Contributor

We have so many packages I don't even remember what we already have :)

Contributor

nox commented Feb 13, 2011

We have so many packages I don't even remember what we already have :)

@yrashk

This comment has been minimized.

Show comment Hide comment
@yrashk

yrashk Feb 13, 2011

Owner

Hehe :) there's a typo in erldis authors, though. Wanna fix?

Owner

yrashk commented Feb 13, 2011

Hehe :) there's a typo in erldis authors, though. Wanna fix?

@nox

This comment has been minimized.

Show comment Hide comment
@nox

nox Feb 13, 2011

Contributor

Typo fixed.

Contributor

nox commented Feb 13, 2011

Typo fixed.

@yrashk

This comment has been minimized.

Show comment Hide comment
@yrashk

yrashk Feb 13, 2011

Owner

It's also in https://github.com/agner/erldis.agner/blob/0.3.2/agner.config, what I usually do is I delete the tag and then fix the typo and then push a new tag. I know this is against what git people say, but those are not typical repos, so I personally shuffle tags if necessary.

Owner

yrashk commented Feb 13, 2011

It's also in https://github.com/agner/erldis.agner/blob/0.3.2/agner.config, what I usually do is I delete the tag and then fix the typo and then push a new tag. I know this is against what git people say, but those are not typical repos, so I personally shuffle tags if necessary.

@nox

This comment has been minimized.

Show comment Hide comment
@nox

nox Feb 13, 2011

Contributor

I don't have strong feelings against that, as you said they are not your regular repos. I may have an idea to avoid that though; I'll write an issue about that later. It's 1:14am here, I'm too tired to fix 0.3.2 sorry :)

Contributor

nox commented Feb 13, 2011

I don't have strong feelings against that, as you said they are not your regular repos. I may have an idea to avoid that though; I'll write an issue about that later. It's 1:14am here, I'm too tired to fix 0.3.2 sorry :)

@yrashk

This comment has been minimized.

Show comment Hide comment
@yrashk

yrashk Feb 13, 2011

Owner

Sure, no problem. If I won't fix it today, feel free to do it later on :) And yeah I am looking forward to that issue

Owner

yrashk commented Feb 13, 2011

Sure, no problem. If I won't fix it today, feel free to do it later on :) And yeah I am looking forward to that issue

@yrashk

This comment has been minimized.

Show comment Hide comment
@yrashk

yrashk Feb 13, 2011

Owner

Speaking about `er', it requires lfe, but it has already been packaged

Owner

yrashk commented Feb 13, 2011

Speaking about `er', it requires lfe, but it has already been packaged

@yrashk

This comment has been minimized.

Show comment Hide comment
@yrashk

yrashk Feb 13, 2011

Owner

I've fixed the typo in erldis 0.3.2

Owner

yrashk commented Feb 13, 2011

I've fixed the typo in erldis 0.3.2

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