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

tools_def.txt Enhancement (Bugzilla Bug 54) #9553

Open
tianocore-issues opened this issue Jul 20, 2016 · 4 comments
Open

tools_def.txt Enhancement (Bugzilla Bug 54) #9553

tianocore-issues opened this issue Jul 20, 2016 · 4 comments
Labels
package:basetools priority:medium Moderate impact. Should be prioritized over lower priority issues. type:feature-request A new feature proposal

Comments

@tianocore-issues
Copy link

This issue was created automatically with bugzilla2github

Bugzilla Bug 54

Date: 2016-07-20T21:10:24+00:00
From: @mdkinney
To: @BobCF
CC: hot.tian, jie.b.gao, sigmaepsilon92

Last updated: 2018-11-05T02:40:02+00:00

@tianocore-issues
Copy link
Author

Comment 79

Date: 2016-07-20 21:10:24 +0000
From: @mdkinney

  • Industry Specification: ---
  • Releases to Fix: EDK II Master
  • Target OS: ---
  • Bugzilla Assignee(s): @BobCF

STLVNUB commented on Apr 9
Hi. Not really an issue, but it is, IMHO

The tools_def file is growing rather large, so I propose to split it into sub TOOL_CHAIN files.
i.e
Make all Tool chains seperate files.
e.g
VS_TOOLS.txt: Contains ALL VS data
GCC_TOOLS.txt: Contains ALL GCC data
ICC_TOOLS.txt: etc
UNIXGCC.txt: etc

Then the tools_def.txt will be quite small and the other files will be easier to manage.

Also
Defining a NASM and MAKE variables for custom building.
e.g building on OS X using GCC49, nasm and make will fail.

_*_NASM_PATH = nasm
change to
*__NASM_PATH = ENV(NASM_PREFIX)/nasm

GCC49MAKE_PATH = DEF(GCC44_IA32_PREFIX)make
change to
_GCC49_MAKE_PATH = make
@lerseklersek added BaseTools Feature labels on Apr 11
@nms42

nms42 commented on Apr 11
Some kind of 'include' (for the benefit of common code) would be good ;-)
@Download-Fritz

Download-Fritz commented on May 14
Michael Zimmermann just submited a patch that allows !include in tools_def. :)

@tianocore-issues
Copy link
Author

Comment 404

Date: 2016-08-03 13:02:17 +0000
From: Michael Zimmermann <>

I actually submitted a patch to split up the tools_def.txt but it wasn't merged yet because there may be problems with the usage model of that file.

here's the thread of the mailing list:
https://www.mail-archive.com/[email protected]/msg12583.html

@tianocore-issues
Copy link
Author

Comment 416

Date: 2016-08-04 20:28:36 +0000
From: Barton <<jie.b.gao>>

Bug Scrub: Assign to base tool owner for investigation

@tianocore-issues
Copy link
Author

Comment 5492

Date: 2018-11-05 02:40:02 +0000
From: Yonghong Zhu <<yonghong.zhu>>

change to Bob

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:basetools priority:medium Moderate impact. Should be prioritized over lower priority issues. type:feature-request A new feature proposal
Projects
None yet
Development

No branches or pull requests

1 participant