Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    0xFireball
    @0xFireball
    that's what the NuGet package was for I believe
    Atsushi Eno
    @atsushieno
    I thought the NuGet package is only about reference?
    Alexander Köplinger
    @akoeplinger
    the nuget package should just have a _._ entry under lib/MonoAndroid which means it'll use the one we ship
    Atsushi Eno
    @atsushieno
    ...if that exists. Which Xamarin.Android are you on? @0xFireball
    Alexander Köplinger
    @akoeplinger
    hmm, looks like the System.Runtime.Loader package doesn't have a MonoAndroid entry under lib/, just netstandard1.5
    which has a typeforwarder to mscorlib, so should be fine
    unless the linker messes with that
    Atsushi Eno
    @atsushieno
    but did we support netstandard1.5 in C7SR1?
    Alexander Köplinger
    @akoeplinger
    @atsushieno that's decided by nuget, afaik it just picks the highest netstandard for our products
    well
    I'm not sure how this works for XA
    0xFireball
    @0xFireball
    would a larger stack trace help? @atsushieno
    Alexander Köplinger
    @akoeplinger
    a diagnostic build log would probably help (run msbuild with /v:diag)
    Atsushi Eno
    @atsushieno
    Yeah, it would show how XA resolved the assemblies
    0xFireball
    @0xFireball
    ok, building again
    do you want the task performance summaries too? I can put it on a gist because it might get quite long
    Alexander Köplinger
    @akoeplinger
    gist is good
    Atsushi Eno
    @atsushieno
    task performance is unnecessary
    @akoeplinger is System.Runtime.Loader in master too? I don't see it in lib/xbuild-frameworks/MonoAndroid/v1.0/Facades
    Alexander Köplinger
    @akoeplinger
    @atsushieno it's not a facade we ship
    the facade should come from the nuget package in this case
    Atsushi Eno
    @atsushieno
    Then I see AssemblyLoaderContext only in mscorlib
    Alexander Köplinger
    @akoeplinger
    yep, that's the correct one
    Atsushi Eno
    @atsushieno
    but who knows that the type marked as in System.Runtime.Loader is in mscorlib? Since the NuGet package does not seem to contain the expected facade, there is nothing that has type forwarder I think.
    Alexander Köplinger
    @akoeplinger
    the nuget package does contain the facade that typeforwards to mscorlib
    the question is if it gets picked :)
    since it's only in the netstandard1.5 folder in lib/
    Atsushi Eno
    @atsushieno
    Then that's indeed a question to @radekdoulik ;-)
    0xFireball
    @0xFireball
    just a quick question: if there are errors like this, is gitter or issues on the repository a better place?
    Atsushi Eno
    @atsushieno
    We don't open github "issues" because we use bugzilla.xamarin.com
    0xFireball
    @0xFireball
    is that a better place or gitter?
    Atsushi Eno
    @atsushieno
    gitter is easier if people are on, but that's not always the case and it also may flow away
    bugzilla is annoying but persistent ;-)
    Alexander Köplinger
    @akoeplinger
    and in cases where it looks like a bug, filing it on bugzilla makes sure it's not forgotten
    0xFireball
    @0xFireball
    should i post the issue on bugzilla? or are you guys aware of it?
    and is it even a bug or just some misconfiguration
    Atsushi Eno
    @atsushieno
    we check bugzilla
    dunno, but you don't have to hesitate, people usually come up with some answer
    (on bugzilla)
    So far it is most likely assigned to our linker guy
    0xFireball
    @0xFireball
    all right, thanks @atsushieno
    0xFireball
    @0xFireball
    i've filed a bug at bugzilla:
    Alexander Köplinger
    @akoeplinger
    thanks!
    0xFireball
    @0xFireball
    any other information needed?
    Atsushi Eno
    @atsushieno
    any further requests should be followed up on bugzilla
    Atsushi Eno
    @atsushieno
    Could this get reviewed and merged? xamarin/xamarin-android#169