Summary

Initial scaffolding for the rust bindings and tests

Review Request #628 — Created April 28, 2021 and submitted April 28, 2021, 9:31 a.m.

Information

Owner

grim

Repository

gplugin/gplugin

Branch

default

Bugs

GPLUGIN-136

Depends On

Commit

36ef3353e832

Reviewers

Groups

pidgin

People

Description

This is a small portion of GPLUGIN-133 to add rust bindings and test that we can load plugins written in rust. The rust portion of this is going to be done by a friend of mine, so to save them time we split it up so that I'm handling the build system side.

Testing Done

ran with meson -Drs=true and verified it failed the unit tests as expected.

Issues

Description	From	Last Updated
I think the option should be rust; we don't have pl for Perl and py for Python.	QuLogic	April 28, 2021, 9:25 a.m.

meson_options.txt (Diff revision 1)

The issue has been dropped. Show all issues

I think the option should be rust; we don't have pl for Perl and py for Python.

grim April 28, 2021, 9:04 a.m.

rust is reserved, just like vala, which we worked around by using vapi. We could name it cargo? I'm open to ideas here.

meson_options.txt:61:0: ERROR: Option name rust is reserved.

QuLogic April 28, 2021, 9:17 a.m.

Ah, I see; maybe rs is fine then. Maybe depends on what it produces; if it's a -sys crate, then maybe rust-sys, if just crates in general, then rust-crate?

grim April 28, 2021, 9:19 a.m.

I've been looking at https://github.com/gtk-rs/gir to get ready for this, I'm not sure if we want to do a sys or api create and would be leaning on people that know more about rust to make that decision.  That said, it looks like we might need the -sys package regardless?

Ship it!

```
Ship It!
```
1. grim April 28, 2021, 9:26 a.m.
  So just going to update it later then if need be?
2. QuLogic April 28, 2021, 9:29 a.m.
  Yea, seems fine.
3. grim April 28, 2021, 9:29 a.m.
  okie dokie!

Status:: Completed