New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update liquidsoap to 2.0.5 #140777
Update liquidsoap to 2.0.5 #140777
Conversation
f2c62e8
to
b1ab38f
Compare
514968b
to
6227d46
Compare
4ce37f8
to
e80cb86
Compare
a7f529a
to
3944357
Compare
That should be all of the libraries documented on https://www.liquidsoap.info/doc-dev/build.html except:
tdsl-image seems pretty abandoned and didn't compile for me |
3944357
to
e9218c7
Compare
568af1a
to
db9301f
Compare
1 package failed to build: 6 packages built: but then when I tried building ligo manually it worked so it was just flaky with nix-review for some weird reason |
db9301f
to
64aa98d
Compare
Thanks for updating this, I tried earlier this year but gave up. I have some more ocalm packages that you might be able to use: master...ehmry:liquidsoap |
This huge PR will be a pain to review. Fortunately many changes are independent and can be reviewed and merged separately. Could you please split this PR into smaller pieces? |
would be glad to see this merged. can you maybe fix the merge conflict and then we could check if anything fails and if not merge it? |
There are unfortunately still things which don't work properly. I've been a
little blocked for time lately but plan on moving some of the more obvious
package upgrades/clean ups to separate prs like suggested before.
Then we can merge liquidsoap with the modules that work and leave the rest
as future work.
I'm not happy with the hacks to get version numbers in as well and would
like to get some feedback from somone with more ocaml experience as to what
the proper solution would be.
…On Thu, 3 Feb 2022, 16:32 Lassulus, ***@***.***> wrote:
would be glad to see this merged. can you maybe fix the merge conflict and
then we could check if anything fails and if not merge it?
—
Reply to this email directly, view it on GitHub
<#140777 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHR4LYBZ3GDKW5VJCRRLHTUZKNYZANCNFSM5FPEK43A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
734cea3
to
3725ff9
Compare
Current status:
it not finding posix-time2 is quite surprising, if anyone has an idea of why please chime in. |
FWIW, |
Result of 5 packages built:
|
Motivation for this change
Liquidsoap no longer provides liquidsoap-full source, so all the savonet libraries must be packaged properly as libraries now.
Camomile's META file did not include a version, which is required for liquidsoap's configure step, so I added it.
Feel free to suggest a better way to accomplish this
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)