Skip to content

DEV: add build from source procedures #1319

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

Merged
merged 9 commits into from
Apr 1, 2025
Merged

Conversation

dwdougherty
Copy link
Collaborator

@dwdougherty dwdougherty commented Mar 25, 2025

DOC-5001

This PR contains build instructions for the platforms listed in this RED ticket. At least one architecture of each has been tested on my personal machine using Docker. I have to test macOS because I really don't want to mess up my main system. I'm looking for a older Mac that I can use.

@dwdougherty dwdougherty added dev CE 8.0 Redis Community Edition 8.0 (future) labels Mar 25, 2025
@dwdougherty dwdougherty requested a review from LiorKogan March 25, 2025 17:04
@dwdougherty dwdougherty self-assigned this Mar 25, 2025
@dwdougherty dwdougherty requested review from a team and adobrzhansky March 25, 2025 17:23
Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few minor issues, maybe, but easily fixed and otherwise language LGTM.

@dwdougherty
Copy link
Collaborator Author

Thank you @LiorKogan, @andy-stark-redis, and @adobrzhansky for the review comments. N.B. I still need to test each of these sets of instructions, but I will go ahead and merge this PR into my feature branch. I'll update as necessary after testing.

- operate
- stack
- oss
linkTitle: Debian 11 (Bullseye)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should just include a guide for the latest version of each of these? to encourage using the latest version? Only bookworm, noble, and rhel/rocky/alma 9? @LiorKogan

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should provide instructions for all the OSes we test against.
They are now going to be listed in the Redis 8.0 release notes

  • Ubuntu 20.04 (Focal Fossa), 22.04 (Jammy Jellyfish), 24.04 (Noble Numbat)
  • CentOS 9
  • Rocky Linux 8.10, 9.5
  • AlmaLinux 8.10, 9.5
  • Debian 11 (Bullseye), 12 (Bookworm)
  • macOS 13 (Ventura), 14 (Sonoma), 15 (Sequoia)

Let me know if we need to fix something

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CentOS 9 wasn't on your list. If I need to add it, I'll need a Dockerfile from @adobrzhansky. Isn't CentOS dead?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dwdougherty dwdougherty merged commit fc80933 into community-edition-8 Apr 1, 2025
5 checks passed
@dwdougherty dwdougherty deleted the DOC-5001 branch April 1, 2025 16:16
dwdougherty added a commit that referenced this pull request Apr 7, 2025
* Edits for CE 8.0

* More edits for CE 8.0

* More edits for CE 8.0

* DEV: module configuration changes for CE8 (#956)

* DEV: CONFIG for modules changes for CE8

* Renaming file

* Apply suggestions from Rachel's code review

Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com>

* Apply suggestions from Rachel's code review (part 2)

Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com>

* Apply suggestions from Lior's code review

* Apply suggestions from Lior's code review (part 2)

---------

Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com>

* DEV: ACL changes for CE8 (#975)

* DEV: ACL changes for CE8

* Applied suggestions from Lior's code review (part 1)

* Applied suggestions from Lior's code review (part 2)

* Fix for something completely unrelated

* Applied suggestions from Lior's code review (part 3)

* async io threads (#1020)

* DEV: update ACL categories for search commands (#1165)

* DEV+OP: add CE8 changes to INFO cmd & Prometheus for RS pages (#1170)

* DEV+OP: add CE8 changes to INFO cmd & Prometheus for RS pages

* Apply review comments.

Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>

* Apply review comments.

---------

Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>

* DEV: collapse cmd categories and remove Stack from layouts and cmd pages (#1180)

* DEV: deprecate dialogs 1, 3, and 4; config. changes (#1197)

* DEV: update modules API docs for CE8 RC1 (#1207)

* DEV: add new HFE commands (#1204)

* DEV: some corrections for the previous merge... (#1211)

* DEV: add new HFE commands

* DEV: correct previous merge

* DEV: update installation docs for CE8 (#1218)

* DEV: initial commit of installation docs for CE8

* DEV: 2nd commit of installation docs for CE8

* DEV: 3rd commit of installation docs for CE8

* Apply suggestions from code review

Thank you, @mich-elle-luna!

Co-authored-by: mich-elle-luna <153109578+mich-elle-luna@users.noreply.github.com>

* Some more patching via to using

* Patch up botched merge text

* Remove more instances of Redis Stack

* Apply code review comments

* Add ACL info related to index creation

* Replace instances of baseurl with relref

* Remove vector-set.md page (prematurely added)

* Minor update to archived installation page

* DEV: add build from source procedures (#1319)

* DEV: add build from source procedures

* Apply review comments; added AlmaLinux and Ubuntu (Focal) pages

* Apply review comments; merged alma/rocky pages; added stubs for macOS

* Added macOS 13/14 page

* Apply code review comments and issues found whilst testing

* Bug fix

* Change titles per Lior

* Remove bash requirement from alma/rocky; it's already there

* Remove banner as instructs are complete

* DEV: point to GitHub for source downloads (#1358)

* DEV: point to GitHub for source downloads

* Adding the remaining pages

* Finalize the build instructions

* DEV: add new vector set data type and command pages (#1334)

* Initial commit of vector set docs

* Add'l commit of vector set docs

* Add'l commit of vector set docs

* Adding data type and command pages

* Added RESP2 and RESP3 return information

* Fix some atrocious grammar on the vs index page.

* A few more quick fixes

* Apply suggestions from code review

Co-authored-by: mich-elle-luna <153109578+mich-elle-luna@users.noreply.github.com>

* Apply review comments

Co-authored-by: mich-elle-luna <153109578+mich-elle-luna@users.noreply.github.com>

* Apply preview banner + misc. fixes

---------

Co-authored-by: mich-elle-luna <153109578+mich-elle-luna@users.noreply.github.com>

* Typo fix

* Update modules-api-ref.md from script

* Update docker installation instructions

* Update brew installation instructions

* Add what's new info

* Update snap installation instructions

---------

Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com>
Co-authored-by: Yuan Wang <wangyuancode@163.com>
Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
Co-authored-by: mich-elle-luna <153109578+mich-elle-luna@users.noreply.github.com>
dwdougherty added a commit that referenced this pull request May 1, 2025
* Edits for CE 8.0

* More edits for CE 8.0

* More edits for CE 8.0

* DEV: module configuration changes for CE8 (#956)

* DEV: CONFIG for modules changes for CE8

* Renaming file

* Apply suggestions from Rachel's code review

Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com>

* Apply suggestions from Rachel's code review (part 2)

Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com>

* Apply suggestions from Lior's code review

* Apply suggestions from Lior's code review (part 2)

---------

Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com>

* DEV: ACL changes for CE8 (#975)

* DEV: ACL changes for CE8

* Applied suggestions from Lior's code review (part 1)

* Applied suggestions from Lior's code review (part 2)

* Fix for something completely unrelated

* Applied suggestions from Lior's code review (part 3)

* async io threads (#1020)

* DEV: update ACL categories for search commands (#1165)

* DEV+OP: add CE8 changes to INFO cmd & Prometheus for RS pages (#1170)

* DEV+OP: add CE8 changes to INFO cmd & Prometheus for RS pages

* Apply review comments.

Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>

* Apply review comments.

---------

Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>

* DEV: collapse cmd categories and remove Stack from layouts and cmd pages (#1180)

* DEV: deprecate dialogs 1, 3, and 4; config. changes (#1197)

* DEV: update modules API docs for CE8 RC1 (#1207)

* DEV: add new HFE commands (#1204)

* DEV: some corrections for the previous merge... (#1211)

* DEV: add new HFE commands

* DEV: correct previous merge

* DEV: update installation docs for CE8 (#1218)

* DEV: initial commit of installation docs for CE8

* DEV: 2nd commit of installation docs for CE8

* DEV: 3rd commit of installation docs for CE8

* Apply suggestions from code review

Thank you, @mich-elle-luna!

Co-authored-by: mich-elle-luna <153109578+mich-elle-luna@users.noreply.github.com>

* Some more patching via to using

* Patch up botched merge text

* Remove more instances of Redis Stack

* Apply code review comments

* Add ACL info related to index creation

* Replace instances of baseurl with relref

* Remove vector-set.md page (prematurely added)

* Minor update to archived installation page

* DEV: add build from source procedures (#1319)

* DEV: add build from source procedures

* Apply review comments; added AlmaLinux and Ubuntu (Focal) pages

* Apply review comments; merged alma/rocky pages; added stubs for macOS

* Added macOS 13/14 page

* Apply code review comments and issues found whilst testing

* Bug fix

* Change titles per Lior

* Remove bash requirement from alma/rocky; it's already there

* Remove banner as instructs are complete

* DEV/OP: Redis Open Source brand changes

* Remove superfluous files

* Update newly added files

* Update newly added files; again

* Update dt>json>ram page

* Update what's new page

* Coupla more updates

* Minor update to product landing page

* Banner and ordering updates

* Apply suggestions from code review

Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>

* Remove macOS 15 build instructions

---------

Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com>
Co-authored-by: Yuan Wang <wangyuancode@163.com>
Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
Co-authored-by: mich-elle-luna <153109578+mich-elle-luna@users.noreply.github.com>
@cmilesb cmilesb mentioned this pull request May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CE 8.0 Redis Community Edition 8.0 (future)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants