Update Rocky Linux 8.7 Build Instructions and Replace .tsh with .sh

[beatreichenbach]

Linked issues

Related discussions

#346
#352
#90
#382

Summarize your change.

Adding documentation for Rocky Linux 8.7 Build.

Describe the reason for the change.

The existing documentation was lacking.

Describe what you have tested and on which operating system.

Rocky Linux 8.7

Add a list of changes, and note any that might need special attention during the review.

If possible, provide screenshots.

[linux-foundation-easycla]

• ✅ login: beatreichenbach / name: Beat Reichenbach (867e4f7, 81a4c52, fd96279)
• ✅ login: pbergeron-adsk (44fb30e, a6688f0, 1ab6aa8)
• ❌ The email address for the commit (f94d65a, 09e7a57, 7b57ff9) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please submit a support request ticket.

[geffrak]

It's great! Thanks for your contribution. I somehow missed it.

[geffrak]

Can you do the other wrappers too?

[beatreichenbach]

Is there any reason for other wrappers? Is there any system that doesn't use sh out of the box?

[geffrak]

I mean rvio.wrapper, py-interp.wrapper, etc.

[beatreichenbach]

@geffrak I think I found all the tcsh scripts. Lmk what you think!

[beatreichenbach]

@geffrak thoughts?

[bernie-laberge]

Hello @beatreichenbach,
I apologize for the super long delay in getting back to you on this !

We would very much like to get this excellent work of yours for the .tcsh with .sh replacement.
Would it be possible to remove the build instructions portion of your PR as I believe it has already been addressed by another PR ?
Namely this file:
docs/build_system/config_linux_rocky8.md

Thank you so much for your contribution !

[bernie-laberge]

This is Fantastic @beatreichenbach !
Thank you so much for your contribution and I apologize for the huge delay.

[beatreichenbach]

@bernie-laberge

Would it be possible to remove the build instructions portion of your PR as I believe it has already been addressed by another PR ? Namely this file: docs/build_system/config_linux_rocky8.md

Sure, I can remove them, however I think it might be worth taking a look at how concise my update was vs what was there before. The previous docs also didn't help me build OpenRV in a docker container (meaning it was reproducible!) as I had to fix a bunch of stuff that wasn't in there.

Requested changes applied

[beatreichenbach]

This is getting really outdated... What's the status here?

[pbergeron-adsk]

Hi @beatreichenbach the team was really focused on other issues but over the next while I'll try to clear up some of the older PRs here, including yours :)

Due to the length of time it has taken, and because we require DCO/EasyCLA, I may end up re-doing your PR myself, but the goal is to include your updates.

[beatreichenbach]

Hi @pbergeron-adsk, thank you for taking this on.
Yeah, just reverting everything that are not wrappers makes sense.

[cedrik-fuoco-adsk]

Hi @beatreichenbach, there are conflicts in all the wrapper because the Qt6 feature branch was merged into main. The conflicts are about using the right OpenSSL based on the VFX platform.

Are you able to fix those conflicts? Otherwise, I can fork your branch and work on the changes myself.
If you are going to work on those changes, could you do the EasyCLA check?

Thank you!

[beatreichenbach]

Are you able to fix those conflicts? Otherwise, I can fork your branch and work on the changes myself. If you are going to work on those changes, could you do the EasyCLA check?

@cedrik-fuoco-adsk Unfortunately I'm unable to do this in the next couple weeks. If you have the bandwidth, please go ahead and take this over. Otherwise I will revisit this sometime in the future.

Thanks!

[cedrik-fuoco-adsk]

This PR has been taken over by #789.

Thank you @beatreichenbach !