{"action":"created","issue":{"url":"https://api.github.com/repos/swig/swig/issues/473","labels_url":"https://api.github.com/repos/swig/swig/issues/473/labels{/name}","comments_url":"https://api.github.com/repos/swig/swig/issues/473/comments","events_url":"https://api.github.com/repos/swig/swig/issues/473/events","html_url":"https://github.com/swig/swig/pull/473","id":96600865,"number":473,"title":"Don't use preinst-swig in the test suite makefile.","user":{"login":"vadz","id":146917,"avatar_url":"https://avatars.githubusercontent.com/u/146917?v=3","gravatar_id":"","url":"https://api.github.com/users/vadz","html_url":"https://github.com/vadz","followers_url":"https://api.github.com/users/vadz/followers","following_url":"https://api.github.com/users/vadz/following{/other_user}","gists_url":"https://api.github.com/users/vadz/gists{/gist_id}","starred_url":"https://api.github.com/users/vadz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vadz/subscriptions","organizations_url":"https://api.github.com/users/vadz/orgs","repos_url":"https://api.github.com/users/vadz/repos","events_url":"https://api.github.com/users/vadz/events{/privacy}","received_events_url":"https://api.github.com/users/vadz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-07-22T15:51:58Z","updated_at":"2015-07-25T13:41:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/swig/swig/pulls/473","html_url":"https://github.com/swig/swig/pull/473","diff_url":"https://github.com/swig/swig/pull/473.diff","patch_url":"https://github.com/swig/swig/pull/473.patch"},"body":"This is unnecessary if we export SWIG_LIB from this makefile instead and prevents SWIGTOOL=gdb from working as gdb can't be used to debug a shell script -- but only the real SWIG binary."},"comment":{"url":"https://api.github.com/repos/swig/swig/issues/comments/124847779","html_url":"https://github.com/swig/swig/pull/473#issuecomment-124847779","issue_url":"https://api.github.com/repos/swig/swig/issues/473","id":124847779,"user":{"login":"vadz","id":146917,"avatar_url":"https://avatars.githubusercontent.com/u/146917?v=3","gravatar_id":"","url":"https://api.github.com/users/vadz","html_url":"https://github.com/vadz","followers_url":"https://api.github.com/users/vadz/followers","following_url":"https://api.github.com/users/vadz/following{/other_user}","gists_url":"https://api.github.com/users/vadz/gists{/gist_id}","starred_url":"https://api.github.com/users/vadz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vadz/subscriptions","organizations_url":"https://api.github.com/users/vadz/orgs","repos_url":"https://api.github.com/users/vadz/repos","events_url":"https://api.github.com/users/vadz/events{/privacy}","received_events_url":"https://api.github.com/users/vadz/received_events","type":"User","site_admin":false},"created_at":"2015-07-25T13:41:35Z","updated_at":"2015-07-25T13:41:35Z","body":"I think requiring to set `SWIG_LIB` is not a big deal, SWIG errors (\"unable to find swig.swg\") are relatively clear in this case. Also, the main point of this patch, for me, is that you won't need to copy-and-paste from the terminal so much because 90% of the time I was doing this in order to run swig under gdb and this can now be done much more easily with `make SWIGTOOL=gdb`.\r\n\r\nAs for getting rid of `preinst-swig` entirely, it's not that simple because of the whole baroque structure of SWIG test suite makefiles `SWIG` variable is being passed around a lot and I don't know how to simply use `env SWIG_LIB=... swig` instead of `preinst-swig` for it...\r\n\r\nI hope you can apply this for now as it does simplify debugging SWIG a lot, and maybe someone (maybe even me, but maybe not) can deal with `preinst-swig` some time later."}}