)]}'
{"tasks/client.yaml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"1696b86e8a09d837977b6a3bd3a8270f908e5b19","unresolved":false,"context_lines":[{"line_number":44,"context_line":"    path: /lib64/libnsl.so.1"},{"line_number":45,"context_line":"  registers: st"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- name: ensure libnsl dependency is installed"},{"line_number":48,"context_line":"  package:"},{"line_number":49,"context_line":"    name: libnsl"},{"line_number":50,"context_line":"    state: present"},{"line_number":51,"context_line":"  when: not st.stat.exists"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"- name: run installer  # noqa 306"},{"line_number":54,"context_line":"  shell: echo \"1\" | /opt/nfast/sbin/install"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_e92a742a","line":51,"range":{"start_line":47,"start_character":0,"end_line":51,"end_character":26},"updated":"2019-08-09 21:47:56.000000000","message":"the package module should be idempotent - it should not e necessary to need the previous stat task?","commit_id":"5d797e3e2a4e8749e98b6b0cce3b9f29159a5cd6"},{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"a480c8a07887509ab282c6f78ba67cd90513f810","unresolved":false,"context_lines":[{"line_number":44,"context_line":"    path: /lib64/libnsl.so.1"},{"line_number":45,"context_line":"  registers: st"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- name: ensure libnsl dependency is installed"},{"line_number":48,"context_line":"  package:"},{"line_number":49,"context_line":"    name: libnsl"},{"line_number":50,"context_line":"    state: present"},{"line_number":51,"context_line":"  when: not st.stat.exists"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"- name: run installer  # noqa 306"},{"line_number":54,"context_line":"  shell: echo \"1\" | /opt/nfast/sbin/install"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_24875164","line":51,"range":{"start_line":47,"start_character":0,"end_line":51,"end_character":26},"in_reply_to":"7faddb67_e92a742a","updated":"2019-08-09 23:22:17.000000000","message":"Good question, Jonathan.  The reason we\u0027re checking for the lib first, is because the package `glibc` used to install the file.  It has been deprecated and removed from `glibc`, and a new package `libnsl` was added for backwards compatibility.\n\nIf the file is there, it means `glibc` includes it, and installing `libnsl` will fail because it does not exist in the distro.\n\nIf the file is not there, it means `glibc` no longer includes it, so `libnsl` needs to be installed.\n\nI should also point out that this is a backport into stable/stein and this patch has already merged into master.","commit_id":"5d797e3e2a4e8749e98b6b0cce3b9f29159a5cd6"}]}
