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

Upgrade generator scripts for downstream packages #299

Closed
16 of 42 tasks
Gnimuc opened this issue May 13, 2021 · 9 comments
Closed
16 of 42 tasks

Upgrade generator scripts for downstream packages #299

Gnimuc opened this issue May 13, 2021 · 9 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@Gnimuc
Copy link
Member

Gnimuc commented May 13, 2021

We should upgrade all of the packages in the list below before releasing v0.14.0:

@Gnimuc Gnimuc added this to the release-0.14.0 milestone May 13, 2021
@Gnimuc Gnimuc added good first issue Good for newcomers help wanted Extra attention is needed labels May 13, 2021
@Gnimuc
Copy link
Member Author

Gnimuc commented May 13, 2021

42, not great, not terrible.

@ankitkumarakt746
Copy link

Hi Gnimuc,
I couldn't understand what kind of upgrade to the packages are you refering to.

@Gnimuc
Copy link
Member Author

Gnimuc commented May 16, 2021

Rewriting those scripts(e.g. this one) to use Clang.jl#master's latest Clang.Generators API: https://github.com/JuliaInterop/Clang.jl#quick-start.

@pazner
Copy link
Contributor

pazner commented Aug 25, 2021

LibCEED has been updated

@melonedo
Copy link
Contributor

ArrowGlib.jl is deprecated in favor of Arrow.jl, I think we do not need to maintain it anymore.

@melonedo
Copy link
Contributor

melonedo commented Aug 28, 2021

H3.jl has already updated its generator.

@Gnimuc
Copy link
Member Author

Gnimuc commented Aug 28, 2021

@melonedo thanks for the hard work! When upgrading those downstream packages, it would be great if all of the pain points can be ameliorated at the Clang.jl's side. For example, instead of doing hacks like
https://github.com/JuliaWeb/LibCURL.jl/pull/105/files#diff-010a0ce3257c161674e34d0844cbaf23d451042222bdff2794ca3836e12cf478R17-R31, we can provide an option for generating enums as constants in the generator.

@odow
Copy link

odow commented Aug 29, 2021

Very nice. Don't let the jump-dev repos block the release. We can take care to update them in the future. They aren't urgent.

@melonedo
Copy link
Contributor

I also updated generator scripts for VideoIO JuliaIO/VideoIO.jl#333.

@Gnimuc Gnimuc removed this from the release-0.14.0 milestone Oct 13, 2021
@Gnimuc Gnimuc pinned this issue Oct 13, 2021
@Gnimuc Gnimuc unpinned this issue Feb 2, 2022
@Gnimuc Gnimuc closed this as completed Oct 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants