Skip to content

fix: correct install paths for shared library and pkg-config file - #825

Merged
gavv merged 1 commit into
roc-streaming:developfrom
baranovmv:fix/scons_install
Aug 24, 2026
Merged

fix: correct install paths for shared library and pkg-config file#825
gavv merged 1 commit into
roc-streaming:developfrom
baranovmv:fix/scons_install

Conversation

@baranovmv

Copy link
Copy Markdown
Member

When installing to system paths, the build was failing with "No such file or directory" errors because AddDistFile was trying to copy from intermediate install locations that didn't exist.

# sudo scons -Q --enable-werror --build-3rdparty=all --disable-pulseaudio --disable-openssl --enable-tests install
  INSTALL /usr/share/man/man1/roc-copy.1 
  INSTALL /usr/share/man/man1/roc-recv.1 
  INSTALL /usr/share/man/man1/roc-send.1 
  INSTALL /usr/lib/x86_64-linux-gnu/libroc.so.0.4 
scons: *** [install] bin/x86_64-pc-linux-gnu/libroc.so.0.4: No such file or directory

@baranovmv
baranovmv requested a review from gavv January 30, 2026 23:54
@rocstreaming-bot

Copy link
Copy Markdown

🤖 Pull request description does not have a link to an issue.
If there is a related issue, please add it to the description using any of the supported formats.

@rocstreaming-bot rocstreaming-bot added the S-ready-for-review status: PR can be reviewed label Jan 30, 2026
@gavv
gavv force-pushed the fix/scons_install branch from 399d0aa to d954e01 Compare June 25, 2026 12:26
@gavv
gavv force-pushed the fix/scons_install branch from d954e01 to 6d6d540 Compare August 24, 2026 12:08
@gavv
gavv force-pushed the fix/scons_install branch from 6d6d540 to 75fb966 Compare August 24, 2026 12:31
@gavv
gavv merged commit e6990c1 into roc-streaming:develop Aug 24, 2026
1 of 47 checks passed
@gavv gavv removed the S-ready-for-review status: PR can be reviewed label Aug 24, 2026
@gavv gavv added this to the 0.5.0 milestone Aug 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