Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

ssdb 1.9.2 #44552

Closed
wants to merge 1 commit into from
Closed

ssdb 1.9.2 #44552

wants to merge 1 commit into from

Conversation

felixbuenemann
Copy link
Sponsor Contributor

  • Update to 1.9.2
  • Patch and install ssdb-cli
  • Separate data directory for ssdb slave
  • Fix ssdb-ins.sh script

* Update to 1.9.2
* Patch and install ssdb-cli
* Separate data directory for ssdb slave
* Fix ssdb-ins.sh script
@dunn
Copy link
Contributor

dunn commented Oct 3, 2015

Have these patches been reported upstream?

@felixbuenemann
Copy link
Sponsor Contributor Author

I don't really think it makes sense, because the patches adjust for the changes in directory structure done by the formula itself. The ssdb-ins.sh could probably be changed upstream to be able to work with both relative and absolute pidfile paths, but that still leaves the difference in the bin path.

@dunn
Copy link
Contributor

dunn commented Oct 3, 2015

Since there's a make step, shouldn't that be configurable?

@felixbuenemann
Copy link
Sponsor Contributor Author

The ssdb-ins.sh file is not generated by make.

@felixbuenemann
Copy link
Sponsor Contributor Author

@dunn I agree that the amount of patching required to get this package to work fine with homebrews directory structure is a bit unhealthy and I'd be preferable to have the upstream build system improved to accommodate for that, but I really don't have the time to do that right now. So maybe we could live with the patches for the time being?

@MikeMcQuaid
Copy link
Member

@dunn Doesn't look too bad to me but feel free to disagree.

@dunn
Copy link
Contributor

dunn commented Oct 18, 2015

@BrewTestBot test this please

@dunn
Copy link
Contributor

dunn commented Oct 18, 2015

Right-o. Merged in 779726a, thank you @felixbuenemann!

@dunn dunn closed this in 779726a Oct 18, 2015
@felixbuenemann
Copy link
Sponsor Contributor Author

Thanks, Alex!

@felixbuenemann felixbuenemann deleted the ssdb-1.9.2 branch October 18, 2015 20:27
@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants