Skip to content
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

test run again with ocamlformat 0.16.0? #441

Open
yuzibo opened this issue Jul 18, 2024 · 9 comments
Open

test run again with ocamlformat 0.16.0? #441

yuzibo opened this issue Jul 18, 2024 · 9 comments

Comments

@yuzibo
Copy link

yuzibo commented Jul 18, 2024

Hi,

I was packaging the bisect-ppx into Debian. But now the ocamlformat was just uploaded there also some days ago, Unfortunately there is only 0.26.2 since. So it seems there is no chance to run test when building, right?

@aantron
Copy link
Owner

aantron commented Jul 24, 2024

Hi! Thank you!

I use prebuilt ocamlformat binaries in Bisect's own CI. I don't know if that could work for testing.

The reason for the specific ocamlformat version is that Bisect needs to normalize its test output across multiple OCaml compiler versions' AST to source dump outputs, so we can run the tests on all those versions, and I just use whatever version of ocamlformat that I found does that correctly. Perhaps I can look into using a more recent version, also, but the result would likely still be pinning ocamlformat to a specific version anyway, as otherwise there is a risk of having to frequently update the tests with ocamlformat profiles, expected outputs, etc.

@aantron
Copy link
Owner

aantron commented Jul 24, 2024

Work for testing in Debian, that is. Can you package a local prebuilt binary of ocamlformat?

@yuzibo
Copy link
Author

yuzibo commented Jul 24, 2024

Hi,

Thank you also.:)

The reason for the specific ocamlformat version is that Bisect needs to normalize its test output across multiple OCaml compiler versions' AST to source dump outputs, so we can run the tests on all those versions, and I just use whatever version of ocamlformat that I found does that correctly. Perhaps I can look into using a more recent version, also, but the result would likely still be pinning ocamlformat to a specific version anyway, as otherwise there is a risk of having to frequently update the tests with ocamlformat profiles, expected outputs, etc.

Understand it. Please notice this issue is just to ask for confirmation and please keep your development workflow.

Work for testing in Debian, that is. Can you package a local prebuilt binary of ocamlformat?

Hmm, I can but there is no help to run tests for bisect-ppx on Debian. The Debian packaging env will fetch all build dependences or tests requirement from Debian official arhives. We can not put one prebuilt binary like ocamlformat into packaging. As we can see here, although ocamlformat has been packaged into Debian, but it's version > 0.26.2 at least.

As I did for bisect-ppx Debian package, I was skipping the test when building and this is okay for Debian. Because I noticed ocamlformat has been packaged recently, so I think I can try to run test for bisect-ppx. So, you know.:)

@aantron
Copy link
Owner

aantron commented Jul 24, 2024

Understood, thank you!

@yuzibo
Copy link
Author

yuzibo commented Jul 24, 2024

Perhaps I can look into using a more recent version, also, but the result would likely still be pinning ocamlformat to a specific version anyway

If this is okay for you also, it would be great to upgrade ocamlformat to 0.26.2 for bisect-ppx then I can versioned it on Debian side. But I can imagine maybe there is lots work that needs to be done.:)

@aantron
Copy link
Owner

aantron commented Jul 24, 2024

I can look into it eventually, but eventually again Debian will update ocamlformat to yet another version, and it seems unreasonable long-term to upgrade ocamlformat in this repo for Debian specifically.

@aantron
Copy link
Owner

aantron commented Jul 24, 2024

As there are other distros, other considerations, etc. It might be best to just continue skipping the tests, unfortunately.

@yuzibo
Copy link
Author

yuzibo commented Jul 24, 2024

I can look into it eventually, but eventually again Debian will update ocamlformat to yet another version, and it seems unreasonable long-term to upgrade ocamlformat in this repo for Debian specifically.

Ah, this could be fixed like below sometime:

debhelper-compat (= 13),

to one specifically version ocamlformat.

As there are other distros, other considerations, etc. It might be best to just continue skipping the tests, unfortunately.

Yeah, I think so too.:)

Please feel free to close the issue.

@aantron
Copy link
Owner

aantron commented Jul 24, 2024

Ah, this could be fixed like below sometime:

debhelper-compat (= 13),

to one specifically version ocamlformat.

Let's leave this open then. I will eventually look into upgrading to 0.26.2, since that version, as I understand it, is and will be available in Debian.

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

No branches or pull requests

2 participants