Merge ~mirespace/ubuntu/+source/augeas:lp-1828074-bionic-augeas-memorybomb into ubuntu/+source/augeas:ubuntu/bionic-devel
Status: | Approved | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Utkarsh Gupta | ||||||||
Approved revision: | b0c6fa20dcc275383ba7dcc3d0ac88ad40aabc38 | ||||||||
Proposed branch: | ~mirespace/ubuntu/+source/augeas:lp-1828074-bionic-augeas-memorybomb | ||||||||
Merge into: | ubuntu/+source/augeas:ubuntu/bionic-devel | ||||||||
Diff against target: |
206 lines (+162/-1) 5 files modified
debian/changelog (+13/-0) debian/control (+2/-1) debian/patches/0005-Replace-pure-funcs-calls-in-path-expr-w-result.patch (+143/-0) debian/patches/series (+1/-0) debian/rules (+3/-0) |
||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Utkarsh Gupta (community) | Approve | ||
Canonical Server MOTU reviewers | Pending | ||
Review via email: mp+408941@code.launchpad.net |
Description of the change
Hi Team,
PPA for these SRUs is : ppa:mirespace/
Two issues were fixed here:
- The use of memory grows as a site conf file enlarges when using augeas:
* d/p/0005-
Replace pure function invocations in
path expressions with their result. Thanks to
David Lutterkort <email address hidden>
(LP: #1828074).
Steps to reproduce the issue are described in the SRU template of the bug.
- Package augeas was FTBFS in Bionic:
* d/rules: Disable gnulib tests, thanks to Matthias
Klose <email address hidden>
(Closes: #919662)(LP: #1813566).
Without this, the package can't be built on Bionic (2 tests fail: test-rwlock1 , test-thread_
Both fixes have been cherry picked from upstream (the patch) and Debian.
Built tests passed ok (No DEP8 tests present):
=======
Testsuite summary for augeas 1.10.1
=======
# TOTAL: 247
# PASS: 243
# SKIP: 4
# XFAIL: 0
# FAIL: 0
# XPASS: 0
# ERROR: 0
=======
Thanks in advance!
Hello,
Thank you for the MP, please find the comments inline. TIA! \o/