{"action":"created","issue":{"url":"https://api.github.com/repos/shedskin/shedskin/issues/58","labels_url":"https://api.github.com/repos/shedskin/shedskin/issues/58/labels{/name}","comments_url":"https://api.github.com/repos/shedskin/shedskin/issues/58/comments","events_url":"https://api.github.com/repos/shedskin/shedskin/issues/58/events","html_url":"https://github.com/shedskin/shedskin/issues/58","id":65261339,"number":58,"title":"Improve the build system","user":{"login":"ShedskinBot","id":11687421,"avatar_url":"https://avatars.githubusercontent.com/u/11687421?v=3","gravatar_id":"","url":"https://api.github.com/users/ShedskinBot","html_url":"https://github.com/ShedskinBot","followers_url":"https://api.github.com/users/ShedskinBot/followers","following_url":"https://api.github.com/users/ShedskinBot/following{/other_user}","gists_url":"https://api.github.com/users/ShedskinBot/gists{/gist_id}","starred_url":"https://api.github.com/users/ShedskinBot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ShedskinBot/subscriptions","organizations_url":"https://api.github.com/users/ShedskinBot/orgs","repos_url":"https://api.github.com/users/ShedskinBot/repos","events_url":"https://api.github.com/users/ShedskinBot/events{/privacy}","received_events_url":"https://api.github.com/users/ShedskinBot/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/shedskin/shedskin/labels/auto-migrated","name":"auto-migrated","color":"ededed"},{"url":"https://api.github.com/repos/shedskin/shedskin/labels/Priority-Low","name":"Priority-Low","color":"ededed"},{"url":"https://api.github.com/repos/shedskin/shedskin/labels/Type-Enhancement","name":"Type-Enhancement","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-03-30T17:31:22Z","updated_at":"2015-03-30T17:31:49Z","closed_at":null,"body":"**Issue by [Jérémie Roquet](/arkanosis)**\n_25 Jan 2010 at 1:54 GMT_\n_Originally opened on Google Code_\n\n----\n\n```\nFor now, compiling a.py that uses b.py produces a Makefile which does:\n\ng++ -O2 -pipe -Wno-deprecated a.cpp shedskin/lib/builtin.cpp b.cpp\nshedskin/lib/re.cpp -o a\n\n1) It'd be better to do\n g++ a.cpp -o a.o\n g++ shedskin/lib/builtin.cpp -o builtin.o\n g++ b.cpp -o b.o\n g++ shedskin/lib/re.cpp -o re.o\n g++ a.o builtin.o b.o re.o -o a\nThen, rebuilding the project would only cause the recompilation of the\nmodified files\n\n2) Actually, builtin.cpp and re.cpp should be compiled when installing shedskin\n\n3) There should be better default options regarding to optimization ie.\n-march=native...\n\n4) Warnings should be activated ie. -Wall -Wextra -std=c++98 -pedantic (at\nleast)\n\n5) Alternate targets for developers would be great:\n - make debug => -O0 -g -ggdb\n - make profile => -O0 -pg -ggdb\n\nPoint 3) requires some benchmarks\nPoint 4) requires some fixes\n\nI add this ticket as a reminder, but I may do it myself someday :-)\n```\n\n\n"},"comment":{"url":"https://api.github.com/repos/shedskin/shedskin/issues/comments/87762936","html_url":"https://github.com/shedskin/shedskin/issues/58#issuecomment-87762936","issue_url":"https://api.github.com/repos/shedskin/shedskin/issues/58","id":87762936,"user":{"login":"ShedskinBot","id":11687421,"avatar_url":"https://avatars.githubusercontent.com/u/11687421?v=3","gravatar_id":"","url":"https://api.github.com/users/ShedskinBot","html_url":"https://github.com/ShedskinBot","followers_url":"https://api.github.com/users/ShedskinBot/followers","following_url":"https://api.github.com/users/ShedskinBot/following{/other_user}","gists_url":"https://api.github.com/users/ShedskinBot/gists{/gist_id}","starred_url":"https://api.github.com/users/ShedskinBot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ShedskinBot/subscriptions","organizations_url":"https://api.github.com/users/ShedskinBot/orgs","repos_url":"https://api.github.com/users/ShedskinBot/repos","events_url":"https://api.github.com/users/ShedskinBot/events{/privacy}","received_events_url":"https://api.github.com/users/ShedskinBot/received_events","type":"User","site_admin":false},"created_at":"2015-03-30T17:31:49Z","updated_at":"2015-03-30T17:31:49Z","body":"**Comment by [Mark Dufour](/srepmub)**\n_4 Apr 2010 at 4:10 GMT_\n\n----\n\n```\nas for point 5, I just added two options to the Makefile. suppose the output \nprogram\nis called 'test', you can now 'make test_prof' and 'make test_debug'.. is that \nabout\nhow you envisioned it? :)\n\n```\n\n\n\n\n"}}