Skip to content

Qt6 - Adding some variable to prepare for Qt6 in openrv-sub (goes into main branch)#672

Merged
cedrik-fuoco-adsk merged 3 commits intoAcademySoftwareFoundation:mainfrom
cedrik-fuoco-adsk:qt6-prep-for-openrv-pub
Feb 4, 2025
Merged

Qt6 - Adding some variable to prepare for Qt6 in openrv-sub (goes into main branch)#672
cedrik-fuoco-adsk merged 3 commits intoAcademySoftwareFoundation:mainfrom
cedrik-fuoco-adsk:qt6-prep-for-openrv-pub

Conversation

@cedrik-fuoco-adsk
Copy link
Copy Markdown
Contributor

Adding some variable to prepare for Qt6 in openrv-sub

Linked issues

n/a

Summarize your change.

Adding some Cmake variables to identify if the Qt package needed is Qt5 or Qt6.

Describe the reason for the change.

This is needed for the future merge of the Qt6 branch but also to push the Qt6 changes to openrv-sub without affecting the current build.

Describe what you have tested and on which operating system.

All

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Copy link
Copy Markdown
Contributor

@bernie-laberge bernie-laberge left a comment

Choose a reason for hiding this comment

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

LGTM

@cedrik-fuoco-adsk cedrik-fuoco-adsk merged commit 17e8191 into AcademySoftwareFoundation:main Feb 4, 2025
kenmcgaugh pushed a commit to kenmcgaugh/OpenRV that referenced this pull request Mar 22, 2025
…o main branch) (AcademySoftwareFoundation#672)

### Adding some variable to prepare for Qt6 in openrv-sub

### Linked issues
n/a

### Summarize your change.
Adding some Cmake variables to identify if the Qt package needed is Qt5
or Qt6.

### Describe the reason for the change.
This is needed for the future merge of the Qt6 branch but also to push
the Qt6 changes to openrv-sub without affecting the current build.

### Describe what you have tested and on which operating system.
All

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Ken McGaugh <ken@mcgaugh.co.uk>
cedrik-fuoco-adsk added a commit to cedrik-fuoco-adsk/OpenRV that referenced this pull request Apr 25, 2025
…o main branch) (AcademySoftwareFoundation#672)

### Adding some variable to prepare for Qt6 in openrv-sub

### Linked issues
n/a

### Summarize your change.
Adding some Cmake variables to identify if the Qt package needed is Qt5
or Qt6.

### Describe the reason for the change.
This is needed for the future merge of the Qt6 branch but also to push
the Qt6 changes to openrv-sub without affecting the current build.

### Describe what you have tested and on which operating system.
All

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
cedrik-fuoco-adsk added a commit to cedrik-fuoco-adsk/OpenRV that referenced this pull request Apr 28, 2025
…o main branch) (AcademySoftwareFoundation#672)

### Adding some variable to prepare for Qt6 in openrv-sub

### Linked issues
n/a

### Summarize your change.
Adding some Cmake variables to identify if the Qt package needed is Qt5
or Qt6.

### Describe the reason for the change.
This is needed for the future merge of the Qt6 branch but also to push
the Qt6 changes to openrv-sub without affecting the current build.

### Describe what you have tested and on which operating system.
All

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
cedrik-fuoco-adsk added a commit to cedrik-fuoco-adsk/OpenRV that referenced this pull request Apr 28, 2025
…o main branch) (AcademySoftwareFoundation#672)

### Adding some variable to prepare for Qt6 in openrv-sub

### Linked issues
n/a

### Summarize your change.
Adding some Cmake variables to identify if the Qt package needed is Qt5
or Qt6.

### Describe the reason for the change.
This is needed for the future merge of the Qt6 branch but also to push
the Qt6 changes to openrv-sub without affecting the current build.

### Describe what you have tested and on which operating system.
All

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
@cedrik-fuoco-adsk cedrik-fuoco-adsk deleted the qt6-prep-for-openrv-pub branch July 28, 2025 13:34
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.

3 participants