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

Crude Tcl interpreter #18

Merged
merged 1 commit into from
Oct 13, 2021
Merged

Crude Tcl interpreter #18

merged 1 commit into from
Oct 13, 2021

Conversation

alain-rs
Copy link
Contributor

Motivate of the pull request

  • To address an existing issue. If so, please provide a link to the issue:
  • [ X] Breaking new feature. If so, please describe details in the description part.

Describe the technical details

What is currently done? (Provide issue link if applicable)

What does this pull request change?

Which part of the code base require a change

  • Library:
  • Plug-in:
  • Engine
  • Documentation
  • Regression tests
  • Continous Integration (CI) scripts

Impact of the pull request

  • Require a change on Quality of Results (QoR)
  • Break back-compatibility. If so, please list who may be influenced.

@tangxifan , I'll need your help with this template..

This is a crude Tcl interpreter embedding in C++.
The build is not completely incremental, the configure part is redone for every compilation. Will try to improve later

Copy link
Contributor

@tangxifan tangxifan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alain-rs Look good to me. We can merge and I will start setting up CI. Just one question here. What is zlib used for here?

@tangxifan
Copy link
Contributor

@alain-rs I will create issue on the missing incremental build feature for the TCL

@alain-rs
Copy link
Contributor Author

zlib is built-in the new Tcl, so it is mandatory

@alain-rs alain-rs merged commit 3f9033e into os-fpga:main Oct 13, 2021
shivaahir1508 pushed a commit that referenced this pull request Jun 15, 2023
* test install

* xml lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* qt lib

* Centos git version

* Undo

* FOEDAG dyn library support

* Change exec name and add device.xml support

* Change exec name and add device.xml support

* Update compiler support

* format

* qt on macos

* Update compiler support

* SDC preprocessor

Co-authored-by: Alain Dargelas <alain.dargelas@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants