-
Notifications
You must be signed in to change notification settings - Fork 247
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
Migrate IceStudion from --board and --top-module flags to apio.ini #769
Comments
thanks @zapta i'm refactoring the toolchain module of Icestudio and i'm taking in mind your improvements in apio. In the next weeks i'm releasing it and this change in your way. I'm maintain this thread open to check when the change will be done in Icestudio. Thanks again! |
Thanks @cavearr. Please be aware of the upcoming changes in apio, the command 'apio init' is going to be replaced in the next release with 'apio create' and 'apio modify'. If you migrate IceStudio before the next apio release, it may be easier for IceStudio to create apio.ini directly as a text file rather than using apio commands. This approach is clean and supported since apio.ini is intended to be maintained and edited by users and compatible with future apio releases. |
Yes, this is the idea, to create apio.init i'm telling you to check it! |
On a second thought, IceStudio writing directly |
Hi @cavearr, some considerations regarding apio.ini mutations and apio make dependencies: Currently apio doesn't track dependencies correctly. For example if the user builds, then modifies the board in apio.ini, and then upload, the upload does not rebuild for the new board. Same with the existing IceStudio unless if you run apio clean each time the user changes the board. This can be fixed in apio by adding apio.ini as a dependency to apio commands such as build, assuming that IceStudio will write apio.ini only when it's really changes, to avoid unnecessary rebuilds. In any case, I am not sure how you plan to migrate IceStudio to use apio.in but whatever you do we will make apio to handle it gracefully. |
Hi @zapta ,i'm very happy to see your implication with apio , go ahead!!! My plan is using all your advances in the integration with Icestudio. I'm now decoupling the toolchain mechanics to do more flexible in general. |
Thanks Carlos!
…On Mon, Sep 23, 2024 at 6:41 AM Carlos Venegas Arrabé < ***@***.***> wrote:
Hi @zapta <https://github.com/zapta> ,i'm very happy to see your
implication with apio , go ahead!!!
My plan is using all your advances in the integration with Icestudio. I'm
now decoupling the toolchain mechanics to do more flexible in general.
—
Reply to this email directly, view it on GitHub
<#769 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAQVMQOYXAJVUSAR3Q47DTLZYALB3AVCNFSM6AAAAABOUVMZYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRYGMYDGMZZG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Finished from 0.13.1w202412210312 wip (https://downloads.icestudio.io) all new apio comands works with apio development toolchain installed. |
Background: When IceStudio invokes apio commands it passes the project configuration using the flags
--board
and--top-module
. These flags has been deprecated and may be deleted in future releases of apio, and the recommended way to convey the project configuration to apio is via a platformio like file calledapio.ini
file that looks like this:This issue is for migrating IceStudio from flags based configuration to
apio.ini
based configuration. Implementation wise, IceStudio can maintain a stableapio.ini
file with the latest project configuration or create anapio.ini
from scratch on each invocation, for example by theapio create
command.For example, with the latter approach, the command
can be translated to this command sequence
Most likely, this will be a local change in the function
apioRun
.icestudio/app/scripts/services/tools.js
Line 209 in 63c6c71
The text was updated successfully, but these errors were encountered: