Bug 998929
Summary: | Change mode in IP address is not accepting it's format | ||
---|---|---|---|
Product: | Red Hat Enterprise Linux 7 | Reporter: | Vladimir Benes <vbenes> |
Component: | NetworkManager | Assignee: | Jirka Klimes <jklimes> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Desktop QE <desktop-qa-list> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 7.0 | CC: | danw, dcbw, pvine, rkhan |
Target Milestone: | beta | ||
Target Release: | 7.0 | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | NetworkManager-0.9.9.0-12.git20130913.el7 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2014-06-13 11:44:50 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Vladimir Benes
2013-08-20 11:06:26 UTC
addresses and routes (and some other properties) differ in value format that is presented while printing and what user actually types. I have modified the 'change' command to present the value to the user in a format that nmcli will accept. Please review the change in upstream jklimes/rh998929-in-out-formats branch. the use of g_regex_split() here is a little confusing, since that's intended to be used when you care about the strings in between the matches, not the matches themselves (eg, splitting on ",\\s*"). But I guess it's less code than the version with g_regex_match() would be, so... It looks like get_property_val() leaks the unconverted property value when it does a conversion. Other than that, everything looks right. (In reply to Dan Winship from comment #3) > the use of g_regex_split() here is a little confusing, since that's intended > to be used when you care about the strings in between the matches, not the > matches themselves (eg, splitting on ",\\s*"). But I guess it's less code > than the version with g_regex_match() would be, so... Yeah, it might be misleading for the first look, but I like how it allows extracting the data quite easily. > It looks like get_property_val() leaks the unconverted property value when > it does a conversion. Other than that, everything looks right. Fixed. Pushed to upstream master: 1f6e1fbc62a6db33dd681faf2b9d8e51093eed2c both enter and backspace + number works well. Covered by both manual and automated tests. This request was resolved in Red Hat Enterprise Linux 7.0. Contact your manager or support representative in case you have further questions about the request. |