Skip to content

[c_compiler] Rename package to native_toolchain_c #87

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

Merged
merged 1 commit into from
Jul 11, 2023

Conversation

dcharkes
Copy link
Collaborator

@dcharkes dcharkes commented Jul 11, 2023

Closes: #85

This will need a manual roll into the Dart SDK to update the package name:

  • pkg/test_runner/lib/src/test_configurations.dart
  • third_party/.gitignore
  • third_party/pkg/c_compiler.status (the file-name itself)
  • tools/bots/test_matrix.json

@github-actions github-actions bot added type-infra A repository infrastructure change or enhancement package:hooks package:hooks_runner labels Jul 11, 2023
@github-actions
Copy link

Package publishing

Package Version Status Publish tag (post-merge)
package:native_toolchain_c 0.1.0 ready to publish native_toolchain_c-v0.1.0
package:native_assets_cli 0.1.0 already published at pub.dev
package:native_assets_builder 0.1.0 already published at pub.dev

Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation.

@coveralls
Copy link

coveralls commented Jul 11, 2023

Coverage Status

coverage: 98.642%. remained the same
when pulling 5ec4cdc on rename-c-compiler
into ca6788d on main.

@dcharkes dcharkes merged commit 1e89ed9 into main Jul 11, 2023
@dcharkes dcharkes deleted the rename-c-compiler branch July 11, 2023 17:00
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Jul 11, 2023
Rolling dart-lang/native#87 manually because
of the c_compiler -> native_toolchain_c rename.

Change-Id: I2592882a7137a40703f96a487a66d31eac7c990d
Cq-Include-Trybots: luci.dart.try:pkg-linux-debug-try,pkg-linux-release-try,pkg-mac-release-arm64-try,pkg-mac-release-try,pkg-win-release-try
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/313200
Commit-Queue: Daco Harkes <[email protected]>
Reviewed-by: Devon Carew <[email protected]>
HosseinYousefi pushed a commit that referenced this pull request Nov 16, 2023
…port. (#87)

* C bindings no longer use fully qualified names. Instead, it uses number-renamed simple names. The classes are sorted in the summarizer to ensure the order is deterministic. This enables shorter symbol names and less verbose bindings.

* C bindings are formatted using same style as Dart SDK, using clang-format. If clang-format is not available, jnigen will issue a warning.
Closes: #84

* C bindings return `JniResult`. The older `Jni.checkException` stopgap is removed. It reduces one line per function binding in dart. Lastly exceptions will work properly on android
Closes: #56
HosseinYousefi pushed a commit that referenced this pull request Nov 16, 2023
…port. (#87)

* C bindings no longer use fully qualified names. Instead, it uses number-renamed simple names. The classes are sorted in the summarizer to ensure the order is deterministic. This enables shorter symbol names and less verbose bindings.

* C bindings are formatted using same style as Dart SDK, using clang-format. If clang-format is not available, jnigen will issue a warning.
Closes: #84

* C bindings return `JniResult`. The older `Jni.checkException` stopgap is removed. It reduces one line per function binding in dart. Lastly exceptions will work properly on android
Closes: #56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:hooks_runner package:hooks type-infra A repository infrastructure change or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[c_compiler] package name
3 participants