{"thread":{"id":"1363","subject":"Makefile: COPTS vs CFLAGS","startedAt":"2005-07-29T13:40:11Z","lastAt":"2005-07-29T14:03:27Z","messageCount":2,"participants":["Petr Baudis","Timo Hirvonen"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"6586","messageId":"20050729134011.GD21909@pasky.ji.cz","threadId":"1363","inReplyTo":null,"subject":"Makefile: COPTS vs CFLAGS","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2005-07-29T13:40:11Z","receivedAt":"2005-07-29T13:40:11Z","isPatch":false,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"  Hello,\n\n  one more thing - should we keep the $COPTS variable? Most projects\n(including Cogito) have the C flags controlled by the $CFLAGS variable.\n\n  What I would propose:\n\n\t-COPTS=-g -O2\n\t-CFLAGS=$(COPTS) -Wall\n\t+CFLAGS?=-g -O2\n\t+CFLAGS+=-Wall\n\n  That is, if user does not specify custom $CFLAGS, default to \"-g -O2\",\nand always append -Wall (and more after I'll send my Makefile patches).\n\n  Opinions?\n\n-- \n\t\t\t\tPetr \"Pasky\" Baudis\nStuff: http://pasky.or.cz/\nIf you want the holes in your knowledge showing up try teaching\nsomeone.  -- Alan Cox\n"},{"id":"6589","messageId":"20050729170327.5f8f3d5a.tihirvon@ee.oulu.fi","threadId":"1363","inReplyTo":"20050729134011.GD21909@pasky.ji.cz","subject":"Re: Makefile: COPTS vs CFLAGS","fromName":"Timo Hirvonen","fromEmail":"tihirvon@ee.oulu.fi","sentAt":"2005-07-29T14:03:27Z","receivedAt":"2005-07-29T14:03:27Z","isPatch":false,"sender":{"key":"tihirvon@gmail.com","avatar":null},"body":"On Fri, 29 Jul 2005 15:40:11 +0200\nPetr Baudis <pasky@suse.cz> wrote:\n\n>   What I would propose:\n> \n> \t-COPTS=-g -O2\n> \t-CFLAGS=$(COPTS) -Wall\n> \t+CFLAGS?=-g -O2\n> \t+CFLAGS+=-Wall\n\nSounds good. I've never heard of anyone using COPTS variable before. And\n\"-g -O2\" seems to be a good default value.\n\nUsing $DESTDIR instead of $dest would be nice too because DESTDIR is used\nby Autotools.\n\n-- \nhttp://onion.dynserv.net/~timo/\n"}]}