Flutter-desktop-embedding: [linux] Makefile doesn't handle spaces in Flutter path

Created on 6 Jul 2019 · 10Comments · Source: google/flutter-desktop-embedding

flutter run is not working for desktop in ubuntu 16.04. I am using a master channel of flutter and clone the git repository of example and when I try "flutter run". I get an error "Build Process failed". The error is related to Makefile.

flutter run
Launching lib/main.dart on Linux in debug mode...
Makefile:107: warning: overriding recipe for target '/home/hunter/Android'
Makefile:100: warning: ignoring old recipe for target '/home/hunter/Android'
Makefile:107: warning: overriding recipe for target 'Developer/Flutter' 
Makefile:100: warning: ignoring old recipe for target 'Developer/Flutter'
Makefile:111: warning: overriding recipe for target '/home/hunter/Android'
Makefile:107: warning: ignoring old recipe for target '/home/hunter/Android'
Makefile:111: warning: overriding recipe for target 'Developer/Flutter' 
Makefile:107: warning: ignoring old recipe for target 'Developer/Flutter'
make: Circular /home/hunter/Android <- /home/hunter/Android dependency dropped.
make: Circular Developer/Flutter <- /home/hunter/Android dependency dropped.
make: Circular Developer/Flutter <- Developer/Flutter dependency dropped.
cp: target                                                              
'Mobile/flutter-desktop-embedding/example/linux/../build/linux/debug/data/icudtl
.dat' is not a directory
make: *** [Developer/Flutter] Error 1                                   
Building Linux application...                                           
Build process failed

os-linux

Source

sachinkum0009

👍2

Most helpful comment

I'm guessing from the output that you have installed flutter in a directory called /home/hunter/Android Developer/Flutter?

Make does very poorly with spaces in paths. We should try to make it handle this if possible, but the simple solution in the short term would be to not have spaces in your installation path. (Generally speaking, it's a bad idea to use spaces in paths on Linux.)

stuartmorgan on 6 Jul 2019

🚀1 ❤1 🎉1 😄1 👍1

All 10 comments

I'm guessing from the output that you have installed flutter in a directory called /home/hunter/Android Developer/Flutter?

stuartmorgan on 6 Jul 2019

🚀1 ❤1 🎉1 😄1 👍1

Thank you for your suggestion I would definitely correct this.

sachinkum0009 on 7 Jul 2019

how

codenoid on 18 Oct 2019

The problem was that there were spaces in the path defined for the flutter. That's why I got up this error.
BTW why did you reopened the issue?

sachinkum0009 on 26 Oct 2019

not me, i'm just askin how was this going, any PR yet ?

codenoid on 26 Oct 2019

@SACHINKUMARHACKER I reopened it because it is a bug that should be fixed if possible, even if there is a workaround.

stuartmorgan on 26 Oct 2019

👍3

Any Idea how can we fix this problem.

sachinkum0009 on 28 Oct 2019

The most robust solution would be to eliminate any possibility of spaces in any variable used in dependency analysis. At least some of the initial errors here should already be fixed now that assemble is putting all the Flutter files we need into flutter/emphemeral, so thing like icudtl.dat are just relative paths.

All use of abspath needs to be replaced, which means among other things a different construction will be needed for objfile paths. That may be tricky to get the rule right for.

stuartmorgan on 28 Oct 2019

👍1

I'm guessing from the output that you have installed flutter in a directory called /home/hunter/Android Developer/Flutter?

Make does very poorly with spaces in paths. We should try to make it handle this if possible, but the simple solution in the short term would be to not have spaces in your installation path. (Generally speaking, it's a bad idea to use spaces in paths on Linux.)

I'm having space in project location and now i solved.