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

Add capability to tp_translator to automatically determine value of CV_DV_UTILS_DIR. #2517

Open
MikeOpenHWGroup opened this issue Jul 29, 2024 · 3 comments
Assignees
Labels
Common Infrastructure Library components or scriptware common to all environments in CORE-V-VERIF tool Issue pertains to a vendor tool bug or incompatibility

Comments

@MikeOpenHWGroup
Copy link
Member

Pull-Request #2511 introduces a new shell environment variable, CV_DV_UTILS_DIR, which the user is required to define in their shell environment. We strive to keep the number of shell variables to a minimum, so our Makefiles typically determine the value of CORE_V_VERIF (example) and then define tool or manifest dependent variables based on that (example).

The purpose of this issue is to add similar functionality to tp_translator.

@MikeOpenHWGroup MikeOpenHWGroup added tool Issue pertains to a vendor tool bug or incompatibility Common Infrastructure Library components or scriptware common to all environments in CORE-V-VERIF labels Jul 29, 2024
@khandelwaltanuj
Copy link
Contributor

Hi @MikeOpenHWGroup

In the DV UTILS we have added the File.f to facilitate the compilation of cv_dv_utils in a new project. It is in this file that the variable CV_DV_UTILS_DIR is used. For the moment I am not sure, how it is possible to do away with this variable.

The example that yu have pointed to is of a makefile but we are not using makefile to compile cv_dv_utils. Hence I am not so sure how it is possible to do away with this variable.

Thanks and Regards
Tanuj Khandelwal

@MikeOpenHWGroup
Copy link
Member Author

The example that yu have pointed to is of a makefile but we are not using makefile to compile cv_dv_utils.

Yes, that is true, but a Python script can do what a Makefile can do (and in a way that is much easier to read!). I suggest that the Python script should do the following:

  • determine the value of CORE_V_VERIF as an absolute (not relative) path
  • define CV_DV_UTILS_DIR using CORE_V_VERIF as the base.

You can look at the Makefile to see an example and it will be similar in Python.

@khandelwaltanuj
Copy link
Contributor

Hi @MikeOpenHWGroup

I am a little confused here. The tp_translator does not use cv_dv_utils_dir.
The cv_dv_utils_dir variable is used in the files list that we have created.

It was needed because an external user cannot user CEA makefile system.

And there is no link between File.f and tp_translator.

Thanks and Regards
Tanuj

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Common Infrastructure Library components or scriptware common to all environments in CORE-V-VERIF tool Issue pertains to a vendor tool bug or incompatibility
Projects
None yet
Development

No branches or pull requests

2 participants