MonoGame / MonoGame.Dependencies

The MonoGame binary dependencies submodule.
38 stars 51 forks source link

Dont use full paths to gtk dlls on linux #40

Closed emarcotte closed 9 years ago

emarcotte commented 9 years ago

To address https://github.com/Mono-Game/MonoGame.Dependencies/issues/39.

Not sure what kind of testing is required, I was able to run the pipeline tool successfully after this change and not prior.

tomspilman commented 9 years ago

@cra0zy @dellis1972 - This seem right to you guys?

Also going to wait till after 3.3 is released to merge this.

harry-cpp commented 9 years ago

This will not work on Ubuntu, this was the first thing I tried when I was setting up config files. I've excluded these files and the dlls from linux installer in https://github.com/mono/MonoGame/pull/3426

If someone could modify Protobuild stuff to not copy gtk and other dll files to the output folder at all, that would be the best fix, because that way the system gtk files would be loaded instead.

tomspilman commented 9 years ago

I've excluded these files and the dlls from linux installer

Will that fix https://github.com/Mono-Game/MonoGame.Dependencies/issues/39 ?

harry-cpp commented 9 years ago

It does fix #39 if people are installing monogame pipeline using the installer. The thing that is not fixed is if the person with non Ubunut linux distro tries to compile from source and just runs the Pipeline. He will either have to delete those few dll files or modify config files.

tomspilman commented 9 years ago

It does fix #39 if people are installing monogame pipeline using the installer

On all Linux distros including Ubuntu then?

The thing that is not fixed is if the person with non Ubunut linux distro tries to compile from source and just runs the Pipeline

So this is Ubuntu specific if working from source code?

Just trying to work out what the issue is and how we need to fix it.

harry-cpp commented 9 years ago

On all Linux distros including Ubuntu then?

Yes

Just trying to work out what the issue is and how we need to fix it.

Here are all the possible ways this issue can be fixed, choose one:

tomspilman commented 9 years ago

Gonna close this... the work @cra0zy is doing will resolve this.