ibrierley / flutter_map_line_editor

A basic line/poly editor that works with flutter_map and dragmarkers.
MIT License
44 stars 25 forks source link

null safety #13

Closed barbalex closed 2 years ago

barbalex commented 3 years ago

So far I have only installed. Successfully. I had to remove flutter_map_dragmarker though because of:

[capturing] flutter pub get
Running "flutter pub get" in capturing...                       
Because every version of flutter_map_line_editor from git depends on flutter_map_dragmarker from git {url: https://github.com/ibrierley/flutter_map_dragmarker.git, ref: nullsafety, path: .} and capturing depends on flutter_map_dragmarker from git {url: https://github.com/ibrierley/flutter_map_dragmarker, ref: nullsafety, path: .}, flutter_map_line_editor from git is forbidden.

So, because capturing depends on flutter_map_line_editor from git, version solving failed.
pub get failed (1; So, because capturing depends on flutter_map_line_editor from git, version solving failed.)
exit code 1

But that will be fine once flutter_map_dragmarker is updated.

ibrierley commented 3 years ago

Thanks for testing, curious I don't get any errors (maybe as I'm running locally, but I do have the same pubspec...are you running the examples or your own test?). If you get bored sometime and want to paste the output of flutter pub deps, I'd be interested, but I suspect it won't be long before I move it to master anyway

barbalex commented 3 years ago

The reason I get it is: I am testing it in my app. I first tested flutter_map_dragmarker, so I added it to pubspec. Then to test flutter_map_line_editor I added that too. That configuration caused this error message. You probably have not done it this way, so you did not run into it.

  # uncommented flutter_map_dragmarker as included in flutter_map_line_editor
  # and flutter does not like a git source being imported twice
  #flutter_map_dragmarker:
  #  git:
  #    url: https://github.com/ibrierley/flutter_map_dragmarker
  #    ref: nullsafety
  flutter_map_line_editor:
    git:
      url: https://github.com/ibrierley/flutter_map_line_editor
      ref: nullsafety

Sorry for not being clear enough.

ibrierley commented 3 years ago

Ok thanks, I think when I update them both to master, the same problem will happen, as there's a double git dependancy. I wasn't aware of that restriction with flutter. Maybe I'll package dragmarker directly in the line_editor as well as standalone. Thanks for testing!

barbalex commented 3 years ago

I think it's all working as designed. I just haven't gotten it to work as desired in my project yet. But that's my own fault as I tried integrating it in an already complex map and I'm pretty sure I have lost the overview of all that is happening 😬.

I will probably have to build a new map from scratch without all the complexity and test it there.

ibrierley commented 2 years ago

I'm closing for this atm, as I think it's not relevant now since latest release a while back, but please reopen/comment if it is!