Bug 2445636 - Review Request: OpenVSP - A parametric aircraft geometry tool
Summary: Review Request: OpenVSP - A parametric aircraft geometry tool
Keywords:
Status: ASSIGNED
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Benson Muite
QA Contact: Fedora Extras Quality Assurance
URL: https://www.openvsp.org
Whiteboard:
Depends On: 2446444 2446445 2446921
Blocks:
TreeView+ depends on / blocked
 
Reported: 2026-03-08 21:50 UTC by marc
Modified: 2026-03-21 12:49 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:
benson_muite: fedora-review?


Attachments (Terms of Use)
The .spec file difference from Copr build 10203972 to 10218154 (2.06 KB, patch)
2026-03-12 20:00 UTC, Fedora Review Service
no flags Details | Diff

Description marc 2026-03-08 21:50:27 UTC
Spec URL: https://download.copr.fedorainfracloud.org/results/mumble07/OpenVSP/fedora-42-x86_64/10203696-OpenVSP/OpenVSP.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/mumble07/OpenVSP/srpm-builds/10203696/OpenVSP-3.47.0-1.src.rpm
Description: OpenVSP is a parametric aircraft geometry tool. OpenVSP allows the user to
create a 3D model of an aircraft defined by common engineering parameters.
This model can be processed into formats suitable for engineering analysis.
Fedora Account System Username: Mumble07

This will be my first submission, and I will require sponsorship.  Assistance would also be appreciated.

I am in contact with Rob McDonald, the developer of OpenVSP.

Comment 1 marc 2026-03-09 07:57:54 UTC
The following built-in dependencies were not packaged into the RPM:
 * libcartesian.so
 * libcfd_mesh.so
 * libgeom_api.so
 * libgeom_api_g.so
 * libgeom_core.so
 * libglfont2.so
 * libgui_and_draw.so
 * librobust.so
 * libsixseries.so
 * libstb_image.so
 * libtritri.so
 * libutil.so
 * libutil_api.so
 * libviewerscreens.so
 * libvsp_graphic.so
 * libvspaero_lib.so
 * libwavedragEL.so
 * libxmlvsp.so

The bundled CMake RPM packager correctly included them.  Is there a way to fix this in the .spec file?

Comment 2 Fedora Review Service 2026-03-09 08:28:16 UTC
Cannot find any valid SRPM URL for this ticket. Common causes are:

- You didn't specify `SRPM URL: ...` in the ticket description
  or any of your comments
- The URL schema isn't HTTP or HTTPS
- The SRPM package linked in your URL doesn't match the package name specified
  in the ticket summary


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 3 marc 2026-03-09 09:13:32 UTC
(In reply to marc from comment #1)
> The following built-in dependencies were not packaged into the RPM:
>  * libcartesian.so
>  * libcfd_mesh.so
>  * libgeom_api.so
>  * libgeom_api_g.so
>  * libgeom_core.so
>  * libglfont2.so
>  * libgui_and_draw.so
>  * librobust.so
>  * libsixseries.so
>  * libstb_image.so
>  * libtritri.so
>  * libutil.so
>  * libutil_api.so
>  * libviewerscreens.so
>  * libvsp_graphic.so
>  * libvspaero_lib.so
>  * libwavedragEL.so
>  * libxmlvsp.so
> 
> The bundled CMake RPM packager correctly included them.  Is there a way to
> fix this in the .spec file?
Fixed, will upload a new .srpm/.spec

Comment 4 marc 2026-03-09 14:05:33 UTC
(In reply to Fedora Review Service from comment #2)
> Cannot find any valid SRPM URL for this ticket. Common causes are:
> 
> - You didn't specify `SRPM URL: ...` in the ticket description
>   or any of your comments
> - The URL schema isn't HTTP or HTTPS
> - The SRPM package linked in your URL doesn't match the package name
> specified
>   in the ticket summary
> 
> 
> ---
> This comment was created by the fedora-review-service
> https://github.com/FrostyX/fedora-review-service
> 
> If you want to trigger a new Copr build, add a comment containing new
> Spec and SRPM URLs or [fedora-review-service-build] string.

Spec URL: https://download.copr.fedorainfracloud.org/results/mumble07/OpenVSP/fedora-42-x86_64/10203696-OpenVSP/OpenVSP.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/mumble07/OpenVSP/srpm-builds/10203696/OpenVSP-3.47.0-1.src.rpm

Comment 5 Fedora Review Service 2026-03-09 14:08:23 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/10203941
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2445636-openvsp/fedora-rawhide-x86_64/10203941-OpenVSP/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 6 marc 2026-03-09 14:20:17 UTC
(In reply to Fedora Review Service from comment #5)
> Copr build:
> https://copr.fedorainfracloud.org/coprs/build/10203941
> (failed)
> 
> Build log:
> https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-
> review-2445636-openvsp/fedora-rawhide-x86_64/10203941-OpenVSP/builder-live.
> log.gz
> 
> Please make sure the package builds successfully at least for Fedora Rawhide.
> 
> - If the build failed for unrelated reasons (e.g. temporary network
>   unavailability), please ignore it.
> - If the build failed because of missing BuildRequires, please make sure they
>   are listed in the "Depends On" field
> 
> 
> ---
> This comment was created by the fedora-review-service
> https://github.com/FrostyX/fedora-review-service
> 
> If you want to trigger a new Copr build, add a comment containing new
> Spec and SRPM URLs or [fedora-review-service-build] string.

Rawhide does not appear to have gcc-14.  The version of STEPCode in OpenVSP does not compile with gcc-15, and is unlikely to compile with gcc-16 (though I haven't checked).  Rawhide compatibility is a bigger ask than f42 and f43 for the moment.  Is that workable for now?

Comment 7 Fedora Review Service 2026-03-09 14:22:48 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/10203972
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2445636-openvsp/fedora-rawhide-x86_64/10203972-OpenVSP/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 8 Benson Muite 2026-03-10 16:29:29 UTC
Is it possible to start by packaging one of the dependencies:
clipper2
delabella
exprparse
libIGES
OpenABF
pinocchio
STEPcode
triangle

Comment 9 marc 2026-03-10 20:31:34 UTC
Sure, but not all of them.  Some are quite a pain to compile, and/or have documented issues when using them as shared libraries.

Comment 10 marc 2026-03-10 22:11:47 UTC
(In reply to Benson Muite from comment #8)
> Is it possible to start by packaging one of the dependencies:
> clipper2
> delabella
> exprparse
> libIGES
> OpenABF
> pinocchio
> STEPcode
> triangle

Would you like these in a separate bug report, or appended to this one?

Comment 13 Fedora Review Service 2026-03-12 20:00:16 UTC
Created attachment 2133111 [details]
The .spec file difference from Copr build 10203972 to 10218154

Comment 14 Fedora Review Service 2026-03-12 20:00:18 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/10218154
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2445636-openvsp/fedora-rawhide-x86_64/10218154-OpenVSP/fedora-review/review.txt

Found issues:

- Not a valid SPDX expression 'NASA-1.3'.
  Read more: https://fedoraproject.org/wiki/Changes/SPDX_Licenses_Phase_1

Please know that there can be false-positives.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.


Note You need to log in before you can comment on or make changes to this bug.