Branch: refs/heads/fix/loading_services_database Home: https://github.com/yast/yast-yast2 Commit: 316699028cbda240f9e2dbce09d479891d60dfbd https://github.com/yast/yast-yast2/commit/316699028cbda240f9e2dbce09d479891d... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/test/port_aliases_test.rb Log Message: ----------- Add missing unit tests The PortAliases public API didn't have unit tests yet, which we considered necessary to make changes in a safety way. Commit: 59a8374b74d0021111f32a1383d6e0039b444a19 https://github.com/yast/yast-yast2/commit/59a8374b74d0021111f32a1383d6e0039b... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb M library/network/test/port_aliases_test.rb Log Message: ----------- Use `getent` to load the services database The `getent` util will load the database properly, no matter if it is placed in /etc/services, /usr/etc/services, or even in both. Commit: 68689097251a6de4a880f80ac0b681de5223cf25 https://github.com/yast/yast-yast2/commit/68689097251a6de4a880f80ac0b681de52... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb M library/network/test/port_aliases_test.rb Log Message: ----------- Improve performance reducing the services lines to process The services database could have almost identical entries for the same services, but with different protocol. Getting rid of the protocol allows to identify and discard those not necessary lines, which helps to improve the performance. Commit: b8ef65895afc3418231cedad626c87671a0a6c83 https://github.com/yast/yast-yast2/commit/b8ef65895afc3418231cedad626c87671a... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb M library/network/test/port_aliases_test.rb Log Message: ----------- Use a Struct to make easier dealing with services Commit: e0a80f5dce7bf7e0318660ac188ba049a04b786c https://github.com/yast/yast-yast2/commit/e0a80f5dce7bf7e0318660ac188ba049a0... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb M library/network/test/port_aliases_test.rb Log Message: ----------- Fix documentation and copyright headers Commit: 278db9c09cc3bbf4008e8258b3ea5d1dd7f19454 https://github.com/yast/yast-yast2/commit/278db9c09cc3bbf4008e8258b3ea5d1dd7... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb M library/network/test/port_aliases_test.rb Log Message: ----------- Get ride of Builtins Commit: a674dc1dd129c3100e8df02e4e31c8d582ef7cab https://github.com/yast/yast-yast2/commit/a674dc1dd129c3100e8df02e4e31c8d582... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M package/yast2.changes M package/yast2.spec Log Message: ----------- Bump version and update changelog Commit: 5c4b61d9e519e3f1211e78e01189117b9feb12d3 https://github.com/yast/yast-yast2/commit/5c4b61d9e519e3f1211e78e01189117b9f... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb Log Message: ----------- Fix Rubocop offenses Commit: 8720d01b3f219f361f8e09bf21d6dbe86a7309c5 https://github.com/yast/yast-yast2/commit/8720d01b3f219f361f8e09bf21d6dbe86a... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb M package/yast2.changes Log Message: ----------- Fix typos and wording based on code review Commit: e23934c964709cd1727977c6062ba9a89bb2e238 https://github.com/yast/yast-yast2/commit/e23934c964709cd1727977c6062ba9a89b... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb M library/network/test/port_aliases_test.rb Log Message: ----------- Avoid using complex output pre-process As suggested in the code review, the performance impact should not be noticiable. Commit: 03577a028701b8e95008604731611125f3d12848 https://github.com/yast/yast-yast2/commit/03577a028701b8e95008604731611125f3... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb Log Message: ----------- Remove pre-loaded service aliases As suggested during the code review. It helps a bit to simplify the code. Commit: 6ab5c6c195f8485f610d988d7c2c47d173a33e8d https://github.com/yast/yast-yast2/commit/6ab5c6c195f8485f610d988d7c2c47d173... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb Log Message: ----------- Remove unused cache Commit: ad5a5ffd3170cd2092dea63da0af59a118c95843 https://github.com/yast/yast-yast2/commit/ad5a5ffd3170cd2092dea63da0af59a118... Author: David Díaz González <dgonzalez@suse.de> Date: 2020-03-06 (Fri, 06 Mar 2020) Changed paths: M library/network/src/modules/PortAliases.rb Log Message: ----------- Do not log `getent sevices` command execution Because its output is too big, now the `logger: nil` option is passed to Yast::Execute in order to use the Cheetah::NullRecorder[1]. [1] https://github.com/openSUSE/cheetah/blob/bafcf46680fba6cc3fa3e60498d2f1d2b74... Compare: https://github.com/yast/yast-yast2/compare/e31523cf6d17...ad5a5ffd3170