Skip to content

Build folder in sketch folder fails on second verify (spurious "build options changed") #8271

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

Closed
czgtest opened this issue Dec 8, 2018 · 3 comments
Labels
Component: CLI The Arduino IDE's command line interface

Comments

@czgtest
Copy link

czgtest commented Dec 8, 2018

Repro on Arduino 1.8.7 and arduino-nightly-windows\arduino-nightly (latest Dec 6 )
OS : windows10

step:
1.download Arduino 1.8.7 or arduino-nightly
2.run verify command line

arduino_debug.exe --verify d:\test\test\1205\111\Blink_error\Blink_error\Blink.ino --board esp8266:esp8266:huzzah:CpuFrequency=80,VTable=flash,FlashSize=4M1M,LwIPVariant=v2mss536,Debug=Disabled,DebugLevel=None____,FlashErase=none,UploadSpeed=115200 --pref build.path=d:\test\test\1205\111\Blink_error\Blink_error\build12

the first time , the verify is pass

image

run verify command in second time , it will fail
image

it always fail due to sketch folder and build.options.json not delete.
image

image

@per1234 per1234 added the Component: CLI The Arduino IDE's command line interface label Dec 8, 2018
@facchinm
Copy link
Member

@czgtest build folder is inside sketch folder, right? We are seeing some similar reports on https://github.com/arduino/arduino-builder ; as a first step, I'll go on and change the title of the issue. In the meantime, make sure to build outside the sketch folder to avoid triggering this bug.

@facchinm facchinm changed the title Arduino verify in command line always fail caused by sketch folder and build.options.json Build folder in sketch folder fails on second verify (spurious "build options changed") Dec 10, 2018
@czgtest
Copy link
Author

czgtest commented Dec 11, 2018

Blink_error.zip

@facchinm attach the folder to you

@facchinm
Copy link
Member

@czgtest I forgot that I already prepared a PR for this error 😄
Here it is arduino/arduino-builder#299

I'm going ahead and merge it so we'll release a build of arduino-builder soonish with this improvement (and it will be available in IDE hourly)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: CLI The Arduino IDE's command line interface
Projects
None yet
Development

No branches or pull requests

3 participants