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

[Discussion] Project abandonment / maintenance #82

Closed
tkroman opened this issue Apr 20, 2017 · 26 comments
Closed

[Discussion] Project abandonment / maintenance #82

tkroman opened this issue Apr 20, 2017 · 26 comments

Comments

@tkroman
Copy link

tkroman commented Apr 20, 2017

Looks like this project is no longer maintained. @kstyrc correct me if I'm wrong.

If that is true, I'd like to start a discussion to see how much people are interested in a possible fork or ownership transfer so the community can take care of the further evolution.

To everyone interested: please voice your opinions on the matter.

@MagnusSmith
Copy link

It would be great to see this continue and get updated. It's so useful

@MagnusSmith
Copy link

Happy to help

@clamey
Copy link

clamey commented May 9, 2017

I'm up for helping as well - looks like there's a bunch of forks:

https://github.com/kstyrc/embedded-redis/network

This one seems to have the most work behind it:

https://github.com/fmonniot/embedded-redis

@tkroman
Copy link
Author

tkroman commented May 9, 2017

Seeing as this hasn't got attention from the author, I think we are free to proceed as we see it fit. I'all do a fork and update to the latest stable redis binary and try to publish. I'll also post an update here to redirect fresh issues to the fork's tracker. Suggestions welcome :)

@clamey
Copy link

clamey commented May 9, 2017

Sounds good

@MagnusSmith
Copy link

Very good

@krzysztof-styrc
Copy link

Sorry for a late reply.

Since long ago I have no time to devote to this project. If you still wanna develop it feel free to fork it. But on the other hand, it would be great to keep it in this repository to guarantee maven repository continuation.

@clamey
Copy link

clamey commented May 10, 2017

@krzysztof-styrc completely understand, life can get busy - thanks for getting the project this far, it's very useful!

I'm kind of leaning to a new fork that's not dependent on a user path and then pointing people to that, but am open to whatever folks want.

@tkroman
Copy link
Author

tkroman commented May 10, 2017

@krzysztof-styrc Did you build cross-platform binaries yourself?

@clamey @MagnusSmith what are your preferred platforms? I have neither resources nor a burning desire to support windows, so this may be a blocker. If someone else thinks they could do it, that's great.

@MagnusSmith
Copy link

There is the Microsoft port of Redis here
One can then use chocolately here to install the latest redis.
This creates a redis-64 directory that includes the redis-server.exe. I have not tested but I'm pretty sure that renaming this with the version suffix should do the job.

@tkroman
Copy link
Author

tkroman commented May 10, 2017

@MagnusSmith yeah, I'm aware of that, but I see the issue in the fact that windows redis binaries may lag behind the latest stable ones, e.g. current stable is 3.2.8 and the MSOpenTech's is 3.2.1. And, as I said, I have no access to the windows machine and all this fuss with VMs may be too much for me as a person who would never use aspect of the library.

@MagnusSmith
Copy link

@tkroman I'm happy to try and support the Windows binaries. It looks like from the RedisExecProvider that we can either attempt a lowest common denominator approach for all platforms or target specific redis versions for different platforms.

@clamey
Copy link

clamey commented May 10, 2017

I don't have ready access to Windows - I'm in Linux and OS X land...

@MagnusSmith
Copy link

@clamey @tkroman I agree that a fork that is not dependent on a user path would be by far the best way to go. Any ideas how we could go about that?
It is hard to see how pull requests could be handled in kstyrc if @krzysztof-styrc is unavailable.

@clamey
Copy link

clamey commented May 10, 2017 via email

@maksymkovalenko
Copy link

It's practical to start small and just upgrade to latest Redis and Java 8.
It's easy and important.

Other suggestions are all good, but can be discussed as the subsequent steps.

@krzysztof-styrc @kstyrc can we start with just this?
Let me know how i can help with this.

@tkroman
Copy link
Author

tkroman commented Jul 13, 2017

So if we find someone who can provide us with windows builds, that would mean we aren't blocked on anything else. I myself can do OS X builds (and linux, with some effort).

@maksymkovalenko
Copy link

i'll do linux

@tkroman
Copy link
Author

tkroman commented Jul 13, 2017

Sooo... windows anyone?

@MagnusSmith
Copy link

MagnusSmith commented Jul 13, 2017 via email

@lucasweb78
Copy link

Has the work discussed above started? Is there a fork that the work is being done on? I Would be interested in both using it and contributing.

@tkroman
Copy link
Author

tkroman commented Aug 7, 2017

@lucasweb78 @MagnusSmith @maksymkovalenko sorry guys, got buried in work. I think I could find some time to do a Mac OS build & perhaps fork a project, if that's really necessary.

@krzysztof-styrc @kstyrc how about distribution? If we did a collective PR with fresh binaries, would you mind releasing an artifact?

Also, guys, WDYT about providing builds for 2.x, 3.x & 4.x simultaneously?

@emjburns
Copy link

@MagnusSmith can you provide a windows binary for Redis 3.2.11? @kstyrc will be merging my PR for 3.2.11 and releasing in a couple of days.

@maksymkovalenko
Copy link

@tkroman I just created PR #110 that adds support for all Redis versions, I added binaries for Linux 64 only (don't have access to Win, Mac OS).
@krzysztof-styrc @kstyrc Please take a look when you have a chance.

@emjburns
Copy link

@maksymkovalenko we forked last year under our github organization (https://github.com/spinnaker/embedded-redis) and are building artifacts/etc. You are welcome to submit PRs to our repo.

@maksymkovalenko
Copy link

Ha-ha, too many new changes on your fork! :) I'll probably have to redo quite a bit of my PR.
But thanks, will rely on this fork.

@tkroman tkroman closed this as not planned Won't fix, can't repro, duplicate, stale May 31, 2024
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

No branches or pull requests

7 participants