Skip to content

Conversation

@Project516
Copy link

@Project516 Project516 commented Sep 23, 2025

I was looking around the repo, and saw the docker file pointed to incorrect debian sources.

@jabrena
Copy link
Member

jabrena commented Sep 26, 2025

Hi @Project516 , try to update this script to understand this PR as a whole:


Rationale for this change

Explain the reasons to change the repository

What changes are included in this PR?

Explain what changes do this PR

Are these changes tested?

Explain if the PR was tested and explain details

Are there any user-facing changes?

Explain if the PR will impact the final user


Note: In the past the builds was compiled in Adoptium but now, we will have to compile in Github actions. What is your big plan to compile in Github Actions?

@Project516
Copy link
Author

Of course,

The changes in this PR are only changing the repository urls from the main debain repo to the archived debian repo, since stretch and buster, since they are end of life. You can see for yourself here.

Yes I tested these changes.

The PR will only affect the users if they are building it.

I did not know this project is moving to Github Actions, and this PR won’t affect anything with that.

Thank you for your time

@jabrena
Copy link
Member

jabrena commented Sep 26, 2025

I need the changes tested in the PR.

@jabrena
Copy link
Member

jabrena commented Sep 29, 2025

I will close this PR and unify the conversation in the another PR. #80

When the complete plan and team will be defined, I will accept this and others PR related to this repository. Any change require an automatic build in a Pipeline, in this case Github Action.

Juan Antonio

@jabrena jabrena closed this Sep 29, 2025
@Project516
Copy link
Author

@jabrena do you plan implement the Github Action Pipeline yourself?

@jabrena
Copy link
Member

jabrena commented Sep 29, 2025

I will review the PRs to approve and merge but I am waiting it from future PR.
It is not only to send a PR to a repository, it is necessary to build it.

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