FSharp.Data.SqlClient icon indicating copy to clipboard operation
FSharp.Data.SqlClient copied to clipboard

SqlProgrammabilityProvider doesnt works on Mono

Open xkrt opened this issue 9 years ago • 8 comments

It seems SqlProgrammabilityProvider doesnt works with Mono: https://twitter.com/mitekm/status/758900343358959617

Related information

  • Linux
  • ver 1.8.2
  • SQL Server 2014 Developer Edition
  • Mono 4.4.2
  • Monodevelop 5.10

xkrt avatar Aug 27 '16 12:08 xkrt

Actually I have that error in compile time: sqlprogrammabilityprovider_arithmetic_overflow

xkrt avatar Aug 29 '16 08:08 xkrt

Did you find a workaround for this? Any response from the maintainers yet?

varon avatar Sep 01 '16 16:09 varon

I have no plans to roll-out Mono support myself. Too much effort because of in-comparability. But PRs are welcome.

dmitry-a-morozov avatar Sep 01 '16 19:09 dmitry-a-morozov

Thanks for the answer, so Mono isn't officially supported, then?

varon avatar Sep 01 '16 19:09 varon

I had this support before for SqlCommandProvider. It lead to ridiculously complex code like

https://github.com/fsprojects/FSharp.Data.SqlClient/blob/f76e5068c13e87a56eebbc205f19afcc6398fb06/src/SqlClient/SqlClientExtensions.fs#L611

https://github.com/fsprojects/FSharp.Data.SqlClient/blob/f76e5068c13e87a56eebbc205f19afcc6398fb06/src/SqlClient/ISqlCommand.fs#L191

and other complications.

Until Mono sql client implementation will follow some basic quality standards I don't think it worth of an effort.

dmitry-a-morozov avatar Sep 01 '16 19:09 dmitry-a-morozov

@dmitry-a-morozov thanks for the answer. It will be good idea to explicitly note in README that "Current Mono not supported". Any plans to support .NET Core?

xkrt avatar Sep 02 '16 05:09 xkrt

@xkrt Yes, there are plans to support .NET Core. But I'll wait until F# compiler will get decent on .NET Core. Speaking of Mono, if they will improve their System.Data.SqlClient implementation I will certainly reconsider. They can do it by taking fully managed implementation that shipped with .NET Core.

dmitry-a-morozov avatar Sep 02 '16 16:09 dmitry-a-morozov

@xkrt if you have a chance, could you check if this issue is still relevant?

smoothdeveloper avatar Apr 17 '19 07:04 smoothdeveloper