{"action":"created","issue":{"url":"https://api.github.com/repos/karelzak/util-linux/issues/208","labels_url":"https://api.github.com/repos/karelzak/util-linux/issues/208/labels{/name}","comments_url":"https://api.github.com/repos/karelzak/util-linux/issues/208/comments","events_url":"https://api.github.com/repos/karelzak/util-linux/issues/208/events","html_url":"https://github.com/karelzak/util-linux/issues/208","id":93500127,"number":208,"title":"readutmp.c and LGPLv2","user":{"login":"atefhaloui","id":13064034,"avatar_url":"https://avatars.githubusercontent.com/u/13064034?v=3","gravatar_id":"","url":"https://api.github.com/users/atefhaloui","html_url":"https://github.com/atefhaloui","followers_url":"https://api.github.com/users/atefhaloui/followers","following_url":"https://api.github.com/users/atefhaloui/following{/other_user}","gists_url":"https://api.github.com/users/atefhaloui/gists{/gist_id}","starred_url":"https://api.github.com/users/atefhaloui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/atefhaloui/subscriptions","organizations_url":"https://api.github.com/users/atefhaloui/orgs","repos_url":"https://api.github.com/users/atefhaloui/repos","events_url":"https://api.github.com/users/atefhaloui/events{/privacy}","received_events_url":"https://api.github.com/users/atefhaloui/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-07T10:34:56Z","updated_at":"2015-07-16T09:39:10Z","closed_at":null,"body":"Hi,\r\n   From my understanding libblkid is licensed under LGPL 2 or higher. However, it depends on the file `readutmp.c` that is licensed under GPLv3.\r\nThe `readutmp` is only required when compiling `lslogins`. Would it be possible to include this file only when required ?\r\n\r\n```diff\r\ndiff -rup util-linux-2.26.2/lib/Makemodule.am util-linux-2.26.2.new/lib/Makemodule.am\r\n--- util-linux-2.26.2/lib/Makemodule.am\t2015-04-29 11:11:51.723818551 +0100\r\n+++ util-linux-2.26.2.new/lib/Makemodule.am\t2015-07-03 16:23:00.697224993 +0100\r\n@@ -24,8 +24,11 @@ libcommon_la_SOURCES = \\\r\n \tlib/sysfs.c \\\r\n \tlib/timeutils.c \\\r\n \tlib/ttyutils.c \\\r\n-\tlib/exec_shell.c \\\r\n-\tlib/readutmp.c\r\n+\tlib/exec_shell.c\r\n+\r\n+if BUILD_LSLOGINS\r\n+libcommon_la_SOURCES += lib/readutmp.c\r\n+endif\r\n \r\n if LINUX\r\n libcommon_la_SOURCES += \\\r\n````\r\n\r\nThank you"},"comment":{"url":"https://api.github.com/repos/karelzak/util-linux/issues/comments/121902427","html_url":"https://github.com/karelzak/util-linux/issues/208#issuecomment-121902427","issue_url":"https://api.github.com/repos/karelzak/util-linux/issues/208","id":121902427,"user":{"login":"karelzak","id":1031987,"avatar_url":"https://avatars.githubusercontent.com/u/1031987?v=3","gravatar_id":"","url":"https://api.github.com/users/karelzak","html_url":"https://github.com/karelzak","followers_url":"https://api.github.com/users/karelzak/followers","following_url":"https://api.github.com/users/karelzak/following{/other_user}","gists_url":"https://api.github.com/users/karelzak/gists{/gist_id}","starred_url":"https://api.github.com/users/karelzak/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karelzak/subscriptions","organizations_url":"https://api.github.com/users/karelzak/orgs","repos_url":"https://api.github.com/users/karelzak/repos","events_url":"https://api.github.com/users/karelzak/events{/privacy}","received_events_url":"https://api.github.com/users/karelzak/received_events","type":"User","site_admin":false},"created_at":"2015-07-16T09:39:10Z","updated_at":"2015-07-16T09:39:10Z","body":"On Tue, Jul 07, 2015 at 03:34:57AM -0700, atefhaloui wrote:\n>    From my understanding libblkid is licensed under LGPL 2 or higher. However, it depends on the file `readutmp.c` that is licensed under GPLv3.\n\nWell, the library is usually installed (by distros) after strip, so\nthe read_utmp() symbol is removed from the library as it's\nunnecessary.\n\n> The `readutmp` is only required when compiling `lslogins`. Would it be possible to include this file only when required ?\n\nThe code is necessary only in lslogins. I have modified and moved the\ncode to lslogins.c.\n\nThanks!\n\n Karel\n\n-- \n Karel Zak  <kzak@redhat.com>\n http://karelzak.blogspot.com\n"}}