Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to compile: avr-gcc.exe: error: unrecognized command line option '-fdiagnostics-color' #445

Closed
denolfe opened this issue Jun 27, 2016 · 4 comments

Comments

@denolfe
Copy link
Contributor

denolfe commented Jun 27, 2016

I pulled the latest changes and I am receiving the following error when attempting to make the default Satan keymap:

/usr/bin/sh: cc: command not found
Compiling: ../../keyboards/satan/satan.c [ERRORS]

avr-gcc.exe: error: unrecognized command line option '-fdiagnostics-color'

../../tmk_core/rules.mk:564: recipe for target '.build/obj_satan_default/keyboards/satan/satan.o' failed
make[1]: *** [.build/obj_satan_default/keyboards/satan/satan.o] Error 1
../../tmk_core/rules.mk:366: recipe for target 'all' failed
make: *** [all] Error 2

After digging, it looks like an else was removed from rules.mk on the pr: 13bb6b4 on line 134. Hope this helps!

@jackhumbert
Copy link
Member

You can disable this temporarily by doing make COLOR=false - it should be excluding it automatically, though. Thanks for reporting!

@jackhumbert
Copy link
Member

Hey, @NoahAndrews - if you get the chance, could you help me figure out what a good way to divert errors to null is on any system? The

ifeq ("$(shell echo "int main(){}" | $(CC) -fdiagnostics-color -x c - -o /dev/null 2>&1)", "")
    CFLAGS+= -fdiagnostics-color
endif

isn't working on Windows like I thought it would.

@NoahAndrews
Copy link
Contributor

To anyone who finds this, use the MHV AVR Tools command line environment instead of the standard Windows one to make colors work correctly. It may also be called Inferno AVR Tools.

@jackhumbert: I think we can close this now

@jackhumbert
Copy link
Member

Sounds good - thanks!

jiaxin96 pushed a commit to Oh-My-Mechanical-Keyboard/qmk_firmware that referenced this issue Oct 18, 2023
* add reviung53 vial files

* Revert "add reviung53 vial files"

This reverts commit 074be536deda760843efda31f0c5ee9dbbb5a5e7.

* add absinthe vial files

* enable lto and disable qmk settings

* set vial tap dance entries

* prettify

* correct formatting

* Remove via specific lines

* Add lighting

* Update keyboards/keyhive/absinthe/keymaps/vial/keymap.c

Co-authored-by: Less/Rikki <[email protected]>

* Update keyboards/keyhive/absinthe/keymaps/vial/rules.mk

Co-authored-by: Less/Rikki <[email protected]>

---------

Co-authored-by: Less/Rikki <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants