{"action":"created","issue":{"url":"https://api.github.com/repos/mxe/mxe/issues/841","labels_url":"https://api.github.com/repos/mxe/mxe/issues/841/labels{/name}","comments_url":"https://api.github.com/repos/mxe/mxe/issues/841/comments","events_url":"https://api.github.com/repos/mxe/mxe/issues/841/events","html_url":"https://github.com/mxe/mxe/issues/841","id":105853604,"number":841,"title":"some packages run wine","user":{"login":"starius","id":920155,"avatar_url":"https://avatars.githubusercontent.com/u/920155?v=3","gravatar_id":"","url":"https://api.github.com/users/starius","html_url":"https://github.com/starius","followers_url":"https://api.github.com/users/starius/followers","following_url":"https://api.github.com/users/starius/following{/other_user}","gists_url":"https://api.github.com/users/starius/gists{/gist_id}","starred_url":"https://api.github.com/users/starius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/starius/subscriptions","organizations_url":"https://api.github.com/users/starius/orgs","repos_url":"https://api.github.com/users/starius/repos","events_url":"https://api.github.com/users/starius/events{/privacy}","received_events_url":"https://api.github.com/users/starius/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-09-10T17:05:36Z","updated_at":"2015-09-24T07:19:53Z","closed_at":null,"body":"Many packages include `configure` step, which can run `wine` (from gtkimageview's log):\r\n\r\n```\r\nconfigure:2945: checking for C compiler default output file name\r\nconfigure:2967: i686-w64-mingw32.static-gcc    conftest.c  >&5\r\nconfigure:2971: $? = 0\r\nconfigure:3009: result: a.exe\r\nconfigure:3026: checking whether the C compiler works\r\nconfigure:3036: ./a.exe\r\nconfigure:3040: $? = 0\r\nconfigure:3057: result: yes\r\nconfigure:3064: checking whether we are cross compiling\r\nconfigure:3066: result: no\r\n```\r\n\r\nOn other machine (without wine):\r\n\r\n```\r\nconfigure:2945: checking for C compiler default output file name\r\nconfigure:2967: i686-w64-mingw32.static-gcc    conftest.c  >&5\r\nconfigure:2971: $? = 0\r\nconfigure:3009: result: a.exe\r\nconfigure:3026: checking whether the C compiler works\r\nconfigure:3036: ./a.exe\r\n./configure: line 3038: ./a.exe: cannot execute binary file: Exec format error\r\nconfigure:3040: $? = 126\r\nconfigure:3057: result: yes\r\nconfigure:3064: checking whether we are cross compiling\r\nconfigure:3066: result: yes\r\n```\r\n\r\nExistence of `wine` affects the value of `whether we are cross compiling`.\r\n\r\nI know that MXE doesn't use wine. Can we force this policy by disabling wine even if it exists when building?"},"comment":{"url":"https://api.github.com/repos/mxe/mxe/issues/comments/142837276","html_url":"https://github.com/mxe/mxe/issues/841#issuecomment-142837276","issue_url":"https://api.github.com/repos/mxe/mxe/issues/841","id":142837276,"user":{"login":"starius","id":920155,"avatar_url":"https://avatars.githubusercontent.com/u/920155?v=3","gravatar_id":"","url":"https://api.github.com/users/starius","html_url":"https://github.com/starius","followers_url":"https://api.github.com/users/starius/followers","following_url":"https://api.github.com/users/starius/following{/other_user}","gists_url":"https://api.github.com/users/starius/gists{/gist_id}","starred_url":"https://api.github.com/users/starius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/starius/subscriptions","organizations_url":"https://api.github.com/users/starius/orgs","repos_url":"https://api.github.com/users/starius/repos","events_url":"https://api.github.com/users/starius/events{/privacy}","received_events_url":"https://api.github.com/users/starius/received_events","type":"User","site_admin":false},"created_at":"2015-09-24T07:19:53Z","updated_at":"2015-09-24T07:19:53Z","body":"I'm checking the build with the following patch:\r\n```diff\r\ndiff --git a/Makefile b/Makefile\r\nindex b0acd09..80c2def 100644\r\n--- a/Makefile\r\n+++ b/Makefile\r\n@@ -48,6 +48,7 @@ PKGS       := $(call set_create,\\\r\n     $(shell $(SED) -n 's/^.* class=\"package\">\\([^<]*\\)<.*$$/\\1/p' '$(TOP_DIR)/index.html'))\r\n BUILD      := $(shell '$(EXT_DIR)/config.guess')\r\n PATH       := $(PREFIX)/$(BUILD)/bin:$(PREFIX)/bin:$(PATH)\r\n+WINEPREFIX := /\r\n \r\n # All pkgs have (implied) order-only dependencies on MXE_CONF_PKGS.\r\n # These aren't meaningful to the pkg list in index.html so\r\n@@ -130,7 +131,7 @@ define MXE_GET_GITHUB_SHA\r\n endef\r\n \r\n # use a minimal whitelist of safe environment variables\r\n-ENV_WHITELIST := PATH LANG MAKE% MXE% %PROXY %proxy LD_LIBRARY_PATH ACLOCAL_PATH\r\n+ENV_WHITELIST := PATH LANG MAKE% MXE% %PROXY %proxy LD_LIBRARY_PATH ACLOCAL_PATH WINEPREFIX\r\n unexport $(filter-out $(ENV_WHITELIST),$(shell env | cut -d '=' -f1))\r\n \r\n SHORT_PKG_VERSION = \\\r\n```\r\n\r\nIs it the right place to add environment variables?"}}