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

REMOVE Ubuntu 20.04 LTS section #4423

Merged
merged 2 commits into from
Sep 11, 2023
Merged

Conversation

fisuda
Copy link
Contributor

@fisuda fisuda commented Sep 10, 2023

This PR removes "Ubuntu 20.04 LTS" section from admin/build_source.md.

When I added Ubuntu 18.04 and Ubuntu 20.04 sections, Orion Context Broker reference distribution was CentOS 7. There are differences between these distributions, so the procedure building Orion for Ubuntu was necessary. After that, the reference distribution was changed to Deiban. The current one is Debian 12.

Currently, the latest version of Ubuntu LTS (Long Term Support) is 22.04. So, I think that Ubuntu 20.04 LTS section should be update to Ubuntu 22.04 LTS section. But, the base distribution of Ubuntu 22.04 LTS is Debian 12. As I show the PR #4422, Orion on Ubuntu 22.04 can be built by the same procedure as Debian 12. Therefore, I thnink that the section to build Orion on Ubuntu is unnecessary.

It would be great if you could review the PR.

Thanks.

@@ -1,6 +1,6 @@
# Building from sources

Orion Context Broker reference distribution is Debian 12. This doesn't mean that the broker cannot be built in other distributions (actually, it can). This section also includes indications on how to build in other distributions, just in the case it may help people that don't use Debian. However, note that the only "officially supported" procedure is the one for Debian 12; the others are provided "as is" and can get obsolete from time to time.
Copy link
Member

Choose a reason for hiding this comment

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

Although after the removal of the Ubuntu section the only procedure would be the one about Debian 12, I'd suggest to keep this fragment just in case in the future another non official procedures would be included, with the following change:

This section also includes indications...

to

This section may include indications...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I updated it.

Copy link
Member

@fgalan fgalan left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@fgalan fgalan merged commit 22af6eb into telefonicaid:master Sep 11, 2023
11 of 12 checks passed
@fisuda fisuda deleted the remove/ubuntu_20_04 branch September 11, 2023 13:07
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

Successfully merging this pull request may close these issues.

2 participants