dart-lang / dart-lang/native

Support multiple libclang versions

Open
#206 0 comments 0 reactions 0 assignees View on GitHub
package:ffigen
Dominant language
Dart
Stars
275
Forks
144
Avg merge
2d 10h
Merged PRs (30d)
47

Description

For a `\`, it might be better to modify the test to accept both, rather than forcing a newer version of clang for all users.

_Originally posted by @dcharkes in https://github.com/dart-lang/native/issues/255

It can be hard getting different libclang versions. See for example:

* https://github.com/flutter/flutter/issues/105513

We should support an as wide range of libclang as possible.

Currently, a single test fails with older versions of libclang due to the single `\`. We should just special-case that test to accept both versions.

* [x] fix test
* [ ] test both libclang 9 and the newest libclang on CI

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.