Skip to content

Conversation

@thesamprice
Copy link

Describe the contribution
A clear and concise description of what the contribution is.

  • Include explicitly what issue it addresses [e.g. Fixes #X]

Adds QT as an operating system.

Testing performed
Steps taken to test the contribution:

  1. Build steps '...'
  2. Execution steps '...'

Built on mac using mac ports install of qt

export PATH=/opt/local/libexec/qt5/bin:$PATH
cmake ..

Expected behavior changes
A clear and concise description of how this contribution will change behavior and level of impact.

  • API Change: xxx (if applicable)
  • Behavior Change: xxx (if applicable)
  • Or no impact to behavior

System(s) tested on

  • Hardware: [Macbook]
  • OS: [qt5]
  • Versions: []

Additional context
Several things not implemented,
Unsure how to implement timers / etc.

Third party code
If included, identify any third party code and provide text file of license
Qt library is linked against.

Contributor Info - All information REQUIRED for consideration of pull request
Full name and company/organization/center of all contributors ("Personal" if individual work)
You have my info.

@skliper
Copy link
Contributor

skliper commented Oct 7, 2021

@thesamprice Could you provide justification for closing? Helps to know it wasn't closed by mistake.

@thesamprice
Copy link
Author

Was unable to compile on my mac due to updates.
Waiting for security to update OS to latest version, and will try again.
Will probably be a long while, as there isn't an immediate driver for me to work it.
Was trying to get it to compile with cfe.
I could keep it open if you think someone else would want to pick it up.

@skliper skliper added the invalid label Oct 7, 2021
@skliper
Copy link
Contributor

skliper commented Oct 7, 2021

@thesamprice Thanks. Marking as invalid for now, can always re-open and change labels if needed.

jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants