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

Packaging: Add new http package for Debian #699

Closed
wants to merge 1 commit into from
Closed

Packaging: Add new http package for Debian #699

wants to merge 1 commit into from

Conversation

SipSeb
Copy link
Contributor

@SipSeb SipSeb commented Jul 7, 2016

The new module contains the http_client and http_async_client module.

A few difficulties with this change:

  1. Modules are right now grouped by lib dependencies. This caused the http_client module to end up in the kamailio-utils-modules package. No one would ever search for it there.
  2. I tried to package a kamailio-http_async-modules package, but this isn't a valid Debian package name. But when renaming it into kamailio-http-modules, it felt wrong to include just the async http client and leave the http_client module stuck in the utils package, where no one would look for it. So I moved the http_client package, too.
  3. The package depends on libevent, which is actually just a dependency of http_async_client. http_client would compile without. I don't know whether this breaks any policies.

The new module contains the http_client and http_async_client
module.
@SipSeb
Copy link
Contributor Author

SipSeb commented Jul 25, 2016

Anything we can do for this pull request? @linuxmaniac @miconda

@miconda
Copy link
Member

miconda commented Jul 27, 2016

IIRC @linuxmaniac added some note in the description of utils package to highlight it has the http client. I am fine with whatever, @linuxmaniac can choose what's easier from maintenance and debian project point of view.

@linuxmaniac
Copy link
Member

the description of kamailio-utils-modules package was updated at 90c9048 and http_async was added to kamailio-extra-modules at 72daa18

@grumvalski
Copy link
Contributor

Thanks a lot Victor!

Cheers,

Federico

On 14 Sep 2016 6:41 pm, "Victor Seva" notifications@github.com wrote:

the description of kamailio-utils-modules package was updated at 90c9048
90c9048
and http_async was added to kamailio-extra-modules at 72daa18
72daa18


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#699 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADip46f9KoWtmhUlLpFK9C3XQVG9smnRks5qqCO8gaJpZM4JHObt
.

@JonHVU
Copy link

JonHVU commented Jul 21, 2017

Hi,

Was this added to 4.4 as I cant see http_async when I install kamailio-extra-modules from;

http://deb.kamailio.org/kamailio44 jessie

Thanks

Jon

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.

None yet

5 participants