gstreamer-rs issue

[Russel Winder]

Hi,

I see gstreamer-rs git HEAD passes it's CI with Rust stable. However, I am now
getting errors such as:


error: no rules expected the token `:`                                                                                                                                
  --> /home/users/russel/.cargo/git/checkouts/gstreamer-rs-79e52a2d27eb91a3/948fb2a/gstreamer/src/auto/bin.rs:28:68                                                  
   |                                                                                                                                                                  
28 |     pub struct Bin(Object<ffi::GstBin, ffi::GstBinClass, BinClass>): Element, Object, ChildProxy;                                                                
   |                                                                    ^                                                                                            
                                                                                                                                                                     
error: no rules expected the token `>`                                                                                                                                
  --> /home/users/russel/.cargo/git/checkouts/gstreamer-rs-79e52a2d27eb91a3/948fb2a/gstreamer/src/auto/buffer_pool.rs:13:77                                          
   |                                                                                                                                                                  
13 |     pub struct BufferPool(Object<ffi::GstBufferPool, ffi::GstBufferPoolClass>): Object;                                                                          
   |                                                                             ^                                                                                    


I have done a "cargo clean; cargo update; cargo build" to get here. I am
hoping there is just something I have missed that will fix this.


--
Russel.
===========================================
Dr Russel Winder      t: [phone]
41 Buckmaster Road    m: [phone]
London SW11 1EN, UK   w: www.russel.org.uk


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[Sebastian Dröge-3]

On Wed, 2019-01-16 at 09:06 +0000, Russel Winder wrote:
> Hi,
>
> I see gstreamer-rs git HEAD passes it's CI with Rust stable. However,
> I am now getting errors such as:
>
> [...]
>
> I have done a "cargo clean; cargo update; cargo build" to get here. I
> am hoping there is just something I have missed that will fix this.

That's because you ran cargo update, which pulled in a newer version of
glib from git. Last night some of my PRs that change parts of the API
were merged but I had no time yet to update gstreamer-rs accordingly.

--
Sebastian Dröge, Centricular Ltd · https://www.centricular.com


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[Russel Winder]

On Wed, 2019-01-16 at 12:10 +0200, Sebastian Dröge wrote:
[…]
>
> That's because you ran cargo update, which pulled in a newer version of
> glib from git. Last night some of my PRs that change parts of the API
> were merged but I had no time yet to update gstreamer-rs accordingly.

Good to know. :-)

I had to do a cargo update because of changes I needed to the dependencies.
Does the git dependency have a simple way of fixing to a commit temporarily to
avoid this sort of problem.

--
Russel.
===========================================
Dr Russel Winder      t: [phone]
41 Buckmaster Road    m: [phone]
London SW11 1EN, UK   w: www.russel.org.uk


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[Sebastian Dröge-3]

On Wed, 2019-01-16 at 10:15 +0000, Russel Winder wrote:

> On Wed, 2019-01-16 at 12:10 +0200, Sebastian Dröge wrote:
> […]
> > That's because you ran cargo update, which pulled in a newer version of
> > glib from git. Last night some of my PRs that change parts of the API
> > were merged but I had no time yet to update gstreamer-rs accordingly.
>
> Good to know. :-)
>
> I had to do a cargo update because of changes I needed to the dependencies.
> Does the git dependency have a simple way of fixing to a commit temporarily to
> avoid this sort of problem.

Yes, you can specify commit ids with the rev attribute/key in the
Cargo.toml.

--
Sebastian Dröge, Centricular Ltd · https://www.centricular.com


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[Russel Winder]

On Wed, 2019-01-16 at 12:33 +0200, Sebastian Dröge wrote:
> […]
>
> Yes, you can specify commit ids with the rev attribute/key in the
> Cargo.toml.
[…]

It seems this is not as simple as it appears at first since all the other gtk-
rs repositories bring in the HEAD dependency of glib. Or I am missing
something (which is very possible).

--
Russel.
===========================================
Dr Russel Winder      t: [phone]
41 Buckmaster Road    m: [phone]
London SW11 1EN, UK   w: www.russel.org.uk


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[Sebastian Dröge-3]

On Wed, 2019-01-16 at 10:50 +0000, Russel Winder wrote:

> On Wed, 2019-01-16 at 12:33 +0200, Sebastian Dröge wrote:
> > […]
> >
> > Yes, you can specify commit ids with the rev attribute/key in the
> > Cargo.toml.
> […]
>
> It seems this is not as simple as it appears at first since all the
> other gtk-
> rs repositories bring in the HEAD dependency of glib. Or I am missing
> something (which is very possible).

Then take this MR :)
https://gitlab.freedesktop.org/gstreamer/gstreamer-rs/merge_requests/201

--
Sebastian Dröge, Centricular Ltd · https://www.centricular.com


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[Russel Winder]

On Wed, 2019-01-16 at 13:52 +0200, Sebastian Dröge wrote:
>
> […]

> Then take this MR :)
> https://gitlab.freedesktop.org/gstreamer/gstreamer-rs/merge_requests/201


I tried just using:

[hidden email]:slomo/gstreamer-rs.git

on the grounds it seems easier than mucking around with merge requests, but I
got:

error: failed to load source for a dependency on `gstreamer`                                                                                                          

Caused by:
  Unable to update /home/users/russel/Repositories/Git/Masters/Private/GStreamer_Rs_Slomo

Caused by:
  found a virtual manifest at `/home/users/russel/Repositories/Git/Masters/Private/GStreamer_Rs_Slomo/Cargo.toml` instead of a package manifest

--
Russel.
===========================================
Dr Russel Winder      t: [phone]
41 Buckmaster Road    m: [phone]
London SW11 1EN, UK   w: www.russel.org.uk


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[Sebastian Dröge-3]

On Wed, 2019-01-16 at 12:17 +0000, Russel Winder wrote:

> On Wed, 2019-01-16 at 13:52 +0200, Sebastian Dröge wrote:
> > […]
> > Then take this MR :)
> > https://gitlab.freedesktop.org/gstreamer/gstreamer-rs/merge_requests/201
>
> I tried just using:
>
> [hidden email]:slomo/gstreamer-rs.git
>
> on the grounds it seems easier than mucking around with merge
> requests, but I got:
> [...]

Just run cargo update, it's merged now :)

--
Sebastian Dröge, Centricular Ltd · https://www.centricular.com


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[Russel Winder]

On Wed, 2019-01-16 at 14:30 +0200, Sebastian Dröge wrote:
[…]
> Just run cargo update, it's merged now :)

I'd got to the stage of really hoping you were going to do that!

Thanks, works a treat.

--
Russel.
===========================================
Dr Russel Winder      t: [phone]
41 Buckmaster Road    m: [phone]
London SW11 1EN, UK   w: www.russel.org.uk


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel