Re: [sr-dev] [kamailio/kamailio] APKBUILD: remove python2 references (PR #3258)

2022-12-08 Thread Henning Westerholt
Merged #3258 into master.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3258#event-7986251429
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] APKBUILD: remove python2 references (PR #3258)

2022-12-08 Thread Henning Westerholt
There seems to be no more progress related to the CI topic discussion. As there 
is no python2 anymore available on alpline, I will merge it. We also excluded 
already the app_python module from newer debian distributions (e.g. bookwork) 
build.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3258#issuecomment-1342258451
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] APKBUILD: remove python2 references (PR #3258)

2022-10-11 Thread sergey-safarov
> Just so I understand correctly: You're referencing the apk binaries that get 
> push to the official Alpine Linux repos, correct? Are you handling this / is 
> any assistance, etc. needed?

No, we push apk to the docker hub image. and then this maybe utilized using 
docker or k8s.

> This issue will probably need to be addressed first (or at least in 
> conjunction) since even if your CI code was correct, build will fail due to 
> inability to provide the depenancies, right?

This will fail on thresh Alpine release. But we implement GitHub actions using 
the old Alpine release with the current packaging script. And then update the 
release and fix python2.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3258#issuecomment-1274918980
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] APKBUILD: remove python2 references (PR #3258)

2022-10-11 Thread Ben Kaufman
> The Alpine automatic build does not work now. Required refactoring after 
> Docker Hub closed the CI-builds on the free accounts. A thing required to 
> write an automatic Alpine build using GitHub actions.
> 
> Let's write alpine automatic builds during the next mouth. If this does not 
> happen, then we can merge.

A few follow up questions on that:

1. Just so I understand correctly:  You're referencing the apk binaries that 
get push to the official Alpine Linux repos, correct?  Are you handling this / 
is any assistance, etc. needed?
2. This issue will probably need to be addressed first (or at least in 
conjunction) since even if your CI code was correct, build will fail due to 
inability to provide the depenancies, right?

With that said, this is far from a critical issue, so i see no reason to rush 
it either.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3258#issuecomment-1274745327
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] APKBUILD: remove python2 references (PR #3258)

2022-10-10 Thread sergey-safarov
The Alpine automatic build does not work now.
Required refactoring after Docker Hub closed the CI-builds on the free accounts.
A thing required to write an automatic Alpine build using GitHub actions.

Let's write alpine automatic builds during the next mouth.
If this does not happen, then we can merge.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3258#issuecomment-1272959786
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] APKBUILD: remove python2 references (PR #3258)

2022-10-10 Thread Henning Westerholt
Thank for the PR - looks good to me. Lets wait a bit for more feedback and then 
merge it.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3258#issuecomment-1272896136
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] APKBUILD: remove python2 references (PR #3258)

2022-10-09 Thread Ben Kaufman
 Pre-Submission Checklist
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, 
...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] No commits to README files for modules (changes must be done to docbook 
files
in `doc/` subfolder, the README file is autogenerated)

 Type Of Change
- [x] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)

 Checklist:
- [x] PR should be backported to stable branches
- [x] Tested changes locally
- [x] Related to issue #3257

 Description
Do not build python2 module with default APKBUILD file due to python2 being 
sunsetted within Alpine Linux (there is no python2 APK).

You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/3258

-- Commit Summary --

  * APKBUILD: remove python2 references

-- File Changes --

M pkg/kamailio/alpine/APKBUILD (12)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/3258.patch
https://github.com/kamailio/kamailio/pull/3258.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3258
You are receiving this because you are subscribed to this thread.

Message ID: kamailio/kamailio/pull/3...@github.com
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev