repl-legacy icon indicating copy to clipboard operation
repl-legacy copied to clipboard

Generate assembly metadata in build script

Open alfonsogarciacaro opened this issue 7 years ago • 7 comments

The metadata for Fable.Core.dll is outdated, we should generate it automatically with every build (and Fable.Import.Browser.dll).

alfonsogarciacaro avatar Jan 15 '18 09:01 alfonsogarciacaro

The "Generate.Metadata" target does that, but it's referencing Fable.Core 1.3.*

<PackageReference Include="Fable.Core" Version="1.3.*" />
<PackageReference Include="Fable.Import.Browser" Version="*" />

Let me know if you want that changed to, say, 1.* or just * for both.

ncave avatar Jan 15 '18 16:01 ncave

Using 1.* could be a good idea, though now that I started 2.0 development, maybe there's no Fable 1.4 after all ;)

alfonsogarciacaro avatar Jan 15 '18 16:01 alfonsogarciacaro

@alfonsogarciacaro I updated the references, so just use the netstandard2.0 binares, performance should be more or less the same.

ncave avatar Jan 15 '18 17:01 ncave

@ncave I've run the Generated.Metadata target (with a fresh clone of your FCS fork). netstandard assemblies are working fine but for some reason I cannot use Fable.Core.jsOptions in the REPL. If it's using the latest Fable.Core that helper should be available. Any hint?

alfonsogarciacaro avatar Jan 15 '18 23:01 alfonsogarciacaro

@alfonsogarciacaro You can open and examine the dll with dnSpy to make sure it's there.

ncave avatar Jan 16 '18 00:01 ncave

@alfonsogarciacaro Looks like it has Fable.Core.JsInterop.jsOptions, but not Fable.Core.jsOptions

ncave avatar Jan 16 '18 00:01 ncave

Slap me in the face please.

Slap me

Actually, Fable.Core.JsInterop was open but the sample was using jsOption (missing s at the end). It works now, sorry for the hustle!

I'm like a mogwai, I shouldn't code after midnight...

alfonsogarciacaro avatar Jan 16 '18 08:01 alfonsogarciacaro