Skip to content

fbc internal changes to track binary tools used by fbc #334

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

Merged
merged 3 commits into from
Aug 7, 2021

Conversation

jayrm
Copy link
Member

@jayrm jayrm commented Aug 7, 2021

This PR expands on ideas discussed in #279 to restructure some fbc internals to use a table of information to track and invoke binary tools - like AS, LD, GCC, etc.

  • name FBCTOOLS_* in the enum FBCTOOL
  • add enum FBCTOOLFLAG to specify tool options
  • add FBCTOOLINFO structure to track additional information for each tool
  • rename toolnames() to fbctoolTB()
  • refactor fbcFindBin() to solve out variables lasttool, last_relying_on_system, and lastpath and store in fbcToolTB()
  • refactor fbcFindBin() to solve out relying_on_system parameter

Intent of the change is to provide some structure for future improvements such as querying the system for available tools or versions of tools, tracking information about tools as it is discovered, or allowing multiple versions / variations of a particular tool.

jayrm added 3 commits August 2, 2021 20:17
- internal changes to prepare for new options on fbc's tools and invocation
- name FBCTOOLS_* in the enum FBCTOOL
- add enum FBCTOOLFLAG to specify tool options
- add FBCTOOLINFO structure to track additional information for each tool
- refactor fbcFindBin() to solve out variables lasttool, last_relying_on_system, and lastpath and store in fbcToolTB()
- refactor fbcFindBin() to solve out relying_on_system parameter
@jayrm jayrm merged commit dfdee71 into freebasic:master Aug 7, 2021
@jayrm jayrm deleted the fbctools branch August 7, 2021 16:45
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

Successfully merging this pull request may close these issues.

1 participant