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

add jami repo #121

Merged
merged 14 commits into from
Feb 22, 2023
Merged

add jami repo #121

merged 14 commits into from
Feb 22, 2023

Conversation

13werwolf13
Copy link
Contributor

today I saw the news about the new release of jami. decided to add it. the only thing I don't understand is how to make a repository for leap. they have it like https://dl.jami.net/nightly/opensuse-leap_15.4/jami-nightly.repo

@13werwolf13
Copy link
Contributor Author

13werwolf13 commented Feb 22, 2023

also found out one problem with jami official repositories. it does not affect the changes in this PR in any way, but without fixing it, it is impossible to check the work on tumbleweed.
https://forum.jami.net/t/tumbleweed-jami-repo-artifact-build-fail/1793
https://git.jami.net/savoirfairelinux/jami-project/-/issues/1546

@13werwolf13 13werwolf13 marked this pull request as ready for review February 22, 2023 07:32
@13werwolf13
Copy link
Contributor Author

@asdil12 I'm not sure if this PR should be completed before the above problem is fixed, but I'm leaning towards it. in any case, it's up to you.

opi/plugins/jami.py Outdated Show resolved Hide resolved
opi/plugins/jami.py Outdated Show resolved Hide resolved
opi/plugins/jami.py Outdated Show resolved Hide resolved
opi/plugins/jami.py Outdated Show resolved Hide resolved
opi/plugins/yandex-browser.py Outdated Show resolved Hide resolved
@asdil12
Copy link
Member

asdil12 commented Feb 22, 2023

According to https://en.wikipedia.org/wiki/Jami_(software) Jami seems to be licensed under GPLv3.
So is there a specific reason why this is not packaged via https://build.opensuse.org?

13werwolf13 and others added 5 commits February 22, 2023 18:49
Co-authored-by: Dominik Heidler <dominik@heidler.eu>
Co-authored-by: Dominik Heidler <dominik@heidler.eu>
Co-authored-by: Dominik Heidler <dominik@heidler.eu>
Co-authored-by: Dominik Heidler <dominik@heidler.eu>
@13werwolf13
Copy link
Contributor Author

According to https://en.wikipedia.org/wiki/Jami_(software) Jami seems to be licensed under GPLv3. So is there a specific reason why this is not packaged via https://build.opensuse.org?

I do not know this. but I think that if there are official repositories from the developer, then it is necessary to give them the opportunity to use them conveniently.

@13werwolf13
Copy link
Contributor Author

checked in container 15.4
everything works as it should

@asdil12 asdil12 merged commit 798fe48 into openSUSE:master Feb 22, 2023
@13werwolf13
Copy link
Contributor Author

it seems that I rushed with these repositories. Based on this thread, they don't have the resources to further maintain rpm repositories for opensuse. if we don't find a solution then the package will move to OBS and I will make a PR to remove this plugin.

@asdil12
Copy link
Member

asdil12 commented Feb 24, 2023

Moving to obs would be the best solution as the packages are then built within a trusted environment and anyone can see the source code that is actually used to build the packages.

If they already built RPM packages in the past, maybe they can provide a spec file that we can reuse.

@13werwolf13
Copy link
Contributor Author

Moving to obs would be the best solution as the packages are then built within a trusted environment and anyone can see the source code that is actually used to build the packages.

If they already built RPM packages in the past, maybe they can provide a spec file that we can reuse.

I have already started to do this, if with their spec I can assemble packages without problems then wait for PR.

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