Skip to content

gst-plugins-imsdk-base: Fix installation of gstreamer-qcom-oss-camera-meta-1.0.pc#1992

Closed
tjamula wants to merge 1 commit intoqualcomm-linux:masterfrom
tjamula:fix_installation_of_camera_base_pkg_config
Closed

gst-plugins-imsdk-base: Fix installation of gstreamer-qcom-oss-camera-meta-1.0.pc#1992
tjamula wants to merge 1 commit intoqualcomm-linux:masterfrom
tjamula:fix_installation_of_camera_base_pkg_config

Conversation

@tjamula
Copy link
Copy Markdown
Contributor

@tjamula tjamula commented Apr 17, 2026

The camera base library is not built, but its pkg-config file was still being installed. Fix patch aligns .pc file installation with library build status.

…-meta-1.0.pc

The camera base library is not built, but its pkg-config file was still being installed.
Fix aligns .pc file installation with library build status.

Signed-off-by: Tushar Patra Jamula <tjamula@qti.qualcomm.com>
actually built. This prevents orphaned .pc files when the library
is excluded from the build via EXCLUDE_FROM_ALL.

Change-Id: I09a495d3db3f47e0b88c08343556f91a9f83b57c
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No Gerrit tags, missing Upstream-Status (wich definitely should be Backport)

Change-Id: I09a495d3db3f47e0b88c08343556f91a9f83b57c
Signed-off-by: Tushar Patra Jamula <tjamula@qti.qualcomm.com>

Upstream-Status: Pending
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why pending? Please send this upstream at least.

@tjamula
Copy link
Copy Markdown
Contributor Author

tjamula commented Apr 24, 2026

The changes are part of the src repo which were merged via internal gerrit and upmerged: gst-plugins-imsdk: qualcomm/gst-plugins-imsdk@631c1e3

The PV also has been upgraded to include the change: #2032

Hence, this PR is rendered moot.

@tjamula tjamula closed this Apr 24, 2026
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