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

Pre Release (v3.9.0-beta.3): Packaging as Rust crate #564

Merged
merged 7 commits into from
May 9, 2023

Conversation

sksat
Copy link
Collaborator

@sksat sksat commented May 9, 2023

概要

c2a-core を Rust の crate としてパッケージングする

詳細

c2a-core は現状あくまでC言語で書かれており,Rustのプロジェクト ではない が,Cargo などのエコシステムだけでも Rust のものに載せる価値は十分にある.
その前準備として,Cargo.toml だけを追加し,crate として公開する.

影響範囲

リリースの手順が変わる.具体的には,バージョン番号をCargo.toml でも管理するようにする.

補足

  • この PR のマージ後,Pre Release でよいのでリリースを打ってほしい(Cargo.toml も更新する必要があるので注意)

備考

  • crate としてリリースするので Pre Release を打つ
  • マージする前にバージョンを上げる

@sksat sksat added enhancement New feature or request priority::high priorityg high labels May 9, 2023
@sksat sksat self-assigned this May 9, 2023
@sksat sksat requested a review from meltingrabbit May 9, 2023 09:24
@meltingrabbit meltingrabbit changed the title Packaging as crate Pre Release (v3.9.0-beta.3): Packaging as crate May 9, 2023
@meltingrabbit
Copy link
Collaborator

PR名 Packaging as Rust crate とかにしたくない?まだこのリポジトリ,Rustのコンテキストないからさ

@sksat
Copy link
Collaborator Author

sksat commented May 9, 2023

閉じはミス?

@meltingrabbit meltingrabbit reopened this May 9, 2023
@meltingrabbit
Copy link
Collaborator

ミス

@meltingrabbit
Copy link
Collaborator

となると,c2a_core_main.h の define って存在意義なくなる?(Cのソースコード上で使いたいこともあるから,ひとまず残すしか無いか)

@sksat
Copy link
Collaborator Author

sksat commented May 9, 2023

あ,PR reopen したら on: pull_request って re-run するんだ

Rust crate,若干不正確ではあるけど気持ちはわかるし,そうしてもまあ別によい

.gitignore Outdated Show resolved Hide resolved
Cargo.toml Outdated Show resolved Hide resolved
@meltingrabbit meltingrabbit changed the title Pre Release (v3.9.0-beta.3): Packaging as crate Pre Release (v3.9.0-beta.3): Packaging as Rust crate May 9, 2023
@sksat
Copy link
Collaborator Author

sksat commented May 9, 2023

となると,c2a_core_main.h の define って存在意義なくなる?(Cのソースコード上で使いたいこともあるから,ひとまず残すしか無いか)

普通にC言語側で使ってるからそんなことはない(ダブって面倒ではある)

@meltingrabbit
Copy link
Collaborator

ちょっと文章変えた

@sksat sksat merged commit 946266b into develop May 9, 2023
@sksat sksat deleted the feature/package-as-crate branch May 9, 2023 16:54
@meltingrabbit
Copy link
Collaborator

リリースうちました: https://github.com/ut-issl/c2a-core/releases/tag/v3.9.0-beta.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority::high priorityg high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants