Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Fixed snapcast etc/default files copy into package - supersedes PR #28/#26#29

Merged
badaix merged 1 commit into
snapcast:masterfrom
davidandreoletti:master
Mar 17, 2024
Merged

Fixed snapcast etc/default files copy into package - supersedes PR #28/#26#29
badaix merged 1 commit into
snapcast:masterfrom
davidandreoletti:master

Conversation

@davidandreoletti

@davidandreoletti davidandreoletti commented Mar 14, 2024

Copy link
Copy Markdown
Contributor

@badaix This PR aims to do all the work for you regarding restoring successful package compilation support on OpenWRT 23.05+ - you contribution is reviewing/merging this please :-)

This PR:

This PR supersedes #26

The instructions in #26 to successfully install the default snapserver/snapclient default config relied on a symlink trick.
On OpenWRT 23.05, the symlink did not resolve to the intended directory. As a result, the package building failed.

This PR supersedes #28

The instructions in #28 allowed one to produce a package without same config files.

This PR solves #23
No comment.

@davidandreoletti davidandreoletti marked this pull request as draft March 14, 2024 07:59
@davidandreoletti davidandreoletti marked this pull request as ready for review March 14, 2024 14:23
@davidandreoletti

davidandreoletti commented Mar 15, 2024

Copy link
Copy Markdown
Contributor Author

@badaix This is good to go and compiles successfully on OpenWRT 23.05 - Tested on x86 OpenWRT.

Would you mind looking at this over the weekend - Won't take more than 6 min.

@badaix badaix merged commit f854566 into snapcast:master Mar 17, 2024
@badaix

badaix commented Mar 17, 2024

Copy link
Copy Markdown
Member

Thanks! Sorry, for the delay.

@davidandreoletti

Copy link
Copy Markdown
Contributor Author

Thank you for taking the time to look at it :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants