SpenceKonde / megaTinyCore

Arduino core for the tinyAVR 0/1/2-series - Ones's digit 2,4,5,7 (pincount, 8,14,20,24), tens digit 0, 1, or 2 (featureset), preceded by flash in kb. Library maintainers: porting help available!
Other
554 stars 144 forks source link

sketch_path used in platform.txt seems to be not longer defined in arduino-cli #935

Closed ArminJo closed 1 year ago

ArminJo commented 1 year ago

sketch_path is used e.g. in line recipe.hooks.savehex.presavehex.1.pattern.linux=cp "{build.path}/{build.project_name}.lst" "{sketch_path}/{build.extraassetname}.lst of platform.txt.

As you can see here, the environment valiable sketch_path sems to be undefined at least in arduino-cli Version: 0.32.2 Commit: 2661f5d9 Date: 2023-04-12T09:41:54Z

Here it worked arduino-cli Version: 0.31.0 Commit: 940c9457 Date: 2023-02-22T16:15:52Z

SpenceKonde commented 1 year ago

What do they say we are to use ro replace it?


Spence Konde Azzy’S Electronics

New products! Check them out at tindie.com/stores/DrAzzy GitHub: github.com/SpenceKonde ATTinyCore: Arduino support for almost every ATTiny microcontroller Contact: @.***

On Mon, Apr 17, 2023, 13:12 Armin @.***> wrote:

As you can see here https://github.com/Arduino-IRremote/Arduino-IRremote/actions/runs/4723699010/jobs/8379998876#step:3:195, the environment valiable sketch_path sems to be undefined at least in arduino-cli Version: 0.32.2 Commit: 2661f5d9 Date: 2023-04-12T09:41:54Z

Here https://github.com/Arduino-IRremote/Arduino-IRremote/actions/runs/4553455603/jobs/8030040602#step:3:63it worked arduino-cli Version: 0.31.0 Commit: 940c9457 Date: 2023-02-22T16:15:52Z

— Reply to this email directly, view it on GitHub https://github.com/SpenceKonde/megaTinyCore/issues/935, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABTXEW2D57QHQSHEZXSJOELXBV2YZANCNFSM6AAAAAAXBPDYBM . You are receiving this because you are subscribed to this thread.Message ID: @.***>

ArminJo commented 1 year ago

I cannot decide, if it is a bug or feature of arduino-cli, but on windows, sketch_path still seems to work. Since I have no linux at home, I do not want to open an issue for arduino-cli.

SpenceKonde commented 1 year ago

I'd say it's a bug in the linux version of the Arduino CLI - How the hell are you supposed to save the hex file if you don't know where to save it to. Please report to the Arduino team, this is a serious bug which would impair usability and maintainability of all three of my cores.

ArminJo commented 1 year ago

Sorry but sketch_path is used to store the .lst and .map files!!! And in ATTinyCore (and all other cores I use for IRremote CI) it gives at least no errorr 😀

per1234 commented 1 year ago

This bug is now tracked by the Arduino CLI developers at https://github.com/arduino/arduino-cli/issues/2340. There is no need to comment there. I'm only adding this comment to notify affected users that there is a formal bug report about it in the appropriate issue tracker.