[sr-dev] [kamailio/kamailio] smsops: Add support for concatenated SMS in decode_3gpp_sms() (#1617)

2018-08-07 Thread Tsvetomir Dimitrov




 Pre-Submission Checklist



- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, 
...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] 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
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [X] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)

 Checklist:

- [ ] PR should be backported to stable branches
- [X] Tested changes locally
- [ ] Related to issue # (replace  with an open issue number)

 Description

Support for concatenated SMS messages in smsops module.

The current implementation doesn't handle correctly concatenated SMSes. The 
patch adds support for them in the smsops module, including dumping a 
concatenated message to the log and new PVs with SMS message parameters.

There are also small fixes in the indentation - spaces are replaces with tabs, 
as this seeems to be the preferred approach in the file.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * smsops: Add support for concatenated SMS in decode_3gpp_sms()

-- File Changes --

M src/modules/smsops/smsops_impl.c (472)

-- Patch Links --

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

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1617
___
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] KEMI: add htable's sht_rm_* functions (#1614)

2018-08-07 Thread Surendra Tiwari
@miconda please assign it to me. i will work on it

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/1614#issuecomment-411132190___
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] KEMI: add htable's sht_rm_* functions (#1614)

2018-08-07 Thread Surendra Tiwari
working on it.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/1614#issuecomment-411132076___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] pkg/kamailio: Defined correct dependences for perl module (#1615) (#1616)

2018-08-07 Thread sergey-safarov
(cherry picked from commit 9ac132ba3a47f13cf7b58839dde34a04348340c9)

User originally reported this for 5.0 branch.
So i created back port for 5.0 branch too 
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * pkg/kamailio: Defined correct dependences for perl module (#1615)

-- File Changes --

M pkg/kamailio/obs/kamailio.spec (4)

-- Patch Links --

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

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1616
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] git:5.1:510a2838: pkg/kamailio: Defined correct dependences for perl module (#1613)

2018-08-07 Thread Victor Seva
Module: kamailio
Branch: 5.1
Commit: 510a2838c3cf226a27b8a88befaefb4fdceae1fd
URL: 
https://github.com/kamailio/kamailio/commit/510a2838c3cf226a27b8a88befaefb4fdceae1fd

Author: Sergey Safarov 
Committer: Victor Seva 
Date: 2018-08-07T12:47:07+02:00

pkg/kamailio: Defined correct dependences for perl module (#1613)

(cherry picked from commit 9ac132ba3a47f13cf7b58839dde34a04348340c9)

---

Modified: pkg/kamailio/obs/kamailio.spec

---

Diff:  
https://github.com/kamailio/kamailio/commit/510a2838c3cf226a27b8a88befaefb4fdceae1fd.diff
Patch: 
https://github.com/kamailio/kamailio/commit/510a2838c3cf226a27b8a88befaefb4fdceae1fd.patch

---

diff --git a/pkg/kamailio/obs/kamailio.spec b/pkg/kamailio/obs/kamailio.spec
index 27744a2f72..d28af186b4 100644
--- a/pkg/kamailio/obs/kamailio.spec
+++ b/pkg/kamailio/obs/kamailio.spec
@@ -567,8 +567,8 @@ Requires:   kamailio = %ver
 Requires:   perl
 BuildRequires:  perl
 %else
-Requires:   mod_perl
-BuildRequires:  mod_perl-devel
+Requires:   perl-libs
+BuildRequires:  perl-ExtUtils-Embed
 %endif
 
 %descriptionperl


___
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] pkg/kamailio: Defined correct dependences for perl module (#1613) (#1615)

2018-08-07 Thread Victor Seva
Merged #1615 into master.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1615#event-1774956962___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] git:master:9ac132ba: pkg/kamailio: Defined correct dependences for perl module (#1613)

2018-08-07 Thread Victor Seva
Module: kamailio
Branch: master
Commit: 9ac132ba3a47f13cf7b58839dde34a04348340c9
URL: 
https://github.com/kamailio/kamailio/commit/9ac132ba3a47f13cf7b58839dde34a04348340c9

Author: Sergey Safarov 
Committer: Victor Seva 
Date: 2018-08-07T12:44:28+02:00

pkg/kamailio: Defined correct dependences for perl module (#1613)

---

Modified: pkg/kamailio/obs/kamailio.spec

---

Diff:  
https://github.com/kamailio/kamailio/commit/9ac132ba3a47f13cf7b58839dde34a04348340c9.diff
Patch: 
https://github.com/kamailio/kamailio/commit/9ac132ba3a47f13cf7b58839dde34a04348340c9.patch

---

diff --git a/pkg/kamailio/obs/kamailio.spec b/pkg/kamailio/obs/kamailio.spec
index cc53dc9e30..c7ba3ce7db 100644
--- a/pkg/kamailio/obs/kamailio.spec
+++ b/pkg/kamailio/obs/kamailio.spec
@@ -578,8 +578,8 @@ Requires:   kamailio = %ver
 Requires:   perl
 BuildRequires:  perl
 %else
-Requires:   mod_perl
-BuildRequires:  mod_perl-devel
+Requires:   perl-libs
+BuildRequires:  perl-ExtUtils-Embed
 %endif
 
 %descriptionperl


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


[sr-dev] [kamailio/kamailio] pkg/kamailio: Defined correct dependences for perl module (#1613) (#1615)

2018-08-07 Thread sergey-safarov
 Pre-Submission Checklist



- [ ] 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
- [ ] Related to issue # (replace  with an open issue number)

 Description
Defined correct dependences for perl module
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * pkg/kamailio: Defined correct dependences for perl module (#1613)

-- File Changes --

M pkg/kamailio/obs/kamailio.spec (4)

-- Patch Links --

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

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1615
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev