-
Notifications
You must be signed in to change notification settings - Fork 458
Extensions.json not generating due to missing trailing slash in NugetPackageRoot #4383
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extensions.json not generating due to missing trailing slash in NugetPackageRoot #4383
Comments
@pgussow -- one thing that we've found is that directly referencing ExtensionsMetadataGenerator does not work correctly. You should remove that and stick with the newest Microsoft.NET.SDK.Functions version (currently 1.0.27) -- which includes the latest ExtensionsMetadataGenerator. |
I grabbed all of your packages, replaced Microsoft.NET.Sdk.Functions with 1.0.27 and I do see the extensions.json appearing... can you try a simple project like this?
|
Does the extensions.json also appear in the correct folder? And does it have the correct content? I've seen that it generates an empty extensions.json as it was not taking the bin folder to examine the DLLs. Also I guess you need to have a function which uses the MessageBus trigger? I've tried a simple project as you've mentioned, but no extensions.json al alll in the build folder. I've attached the simple project I used. |
I too am having this issue. I've spent about 3 hours reading all the past issues w.r.t extensions.json being empty or not generated. I've tried combinations of _IsFunctionsSdkBuild true/false with and without Microsoft.Azure.WebJobs.Script.ExtensionsMetadataGenerator (1.1.0). i either generate an empty extensions.json or no extension.json at all. I uninstalled all .Net Core SDK's and Runtimes, and reinstalled VS 2019 Preview. I cleared all my nuget caches. Simple HTTP triggers work ( no extensions ). As soon as I add "Microsoft.Azure.WebJobs.Extensions.Storage" and try to use a QueueTrigger:
My project file:
I made my own extensions.json:
If i stick this in ./bin/Debug/netcoreapp2.2/ (where the host.json is), it still won't work. If however if i move it into ./bin/Debug/netcoreapp2.2/bin/
then it will work. |
I'm trying to get this to repro and every time I do it, I'm getting extensions.json generated. Let's see what else could be different:
Another option is to run your build with the |
Mine is a bare-bones VS 2019 preview 3.0 project and build ( F5 ).
|
@foconnor-DS -- what about when you go to the directory and do a |
Cleared bin/obj, ran a nuget command line, and then msbuild with binlog. No change, sending you that binlog now. |
We figured out @foconnor-DS's issue -- it had to do with the @pgussow -- can you check that in your setup? The binlog made it pretty clear when I looked at it. I saw a log like this:
|
Okay -- we're getting somewhere. Can you share your exact project file and command you're running to build? |
I've sent you the binlog and the exact project file. The command I run is either |
I believe this isn't working because of the direct reference to ExtensionsMetadataGenerator. I'm working on a fix for that, but if you remove that nuget package, I believe it will all work for you. |
I removed the direct reference. Then the extensions.json gets generated in the proper folder (because of the trailing \ in the NuGetPackageRoot. But if I remove the |
https://www.nuget.org/packages/Microsoft.NET.Sdk.Functions/1.0.28 has been released, which should fix the "missing trailing backslash" issue on the @pgussow -- if you're still seeing issues with this package, can you send me another binlog of a failed build? You shouldn't need the |
So far so good. I don't see any issues anymore. |
1.0.28 changed nothing for me. But after I uninstalled all sdks except one 2.2.204, the error gone. |
@enternet that's interesting. If you are able to reproduce this again, could you try running with a |
OK, lets wait the next sdk. I'm not sure that my case is common case. I had more then 50 sdks. |
Forking an issue reported here to a new issue.
Copying relevant comments from @pgussow here:
The text was updated successfully, but these errors were encountered: