-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Component Licensing] SDK needs to make sure that licenses.licx ends up as "EmbeddedResource" #3631
Comments
License.licx can live in both the AppDesigner folder (which can be renamed to something custom) and in the root of the project. tag @nguerrera |
@livarcocc Can you put this in the milestone that corresponds with 16.4 as per offline agreement? |
@vatsan-madhavan @dsplaisted should this globbing go in WindowsDesktop sdk? |
@davkean I’ve put it in 3.1.1xx milestone, which matches 16.4. Thank you for the detailed issue. |
Sure, that’s ok by me. What’s the plan for LC.exe? Wouldn’t we want a netcoreapp version of it as well? |
@vatsan-madhavan LC.exe executes code so controls can generate the license data you are embedding (you are not embedding the literal licx file, its just instructions for LC.exe which controls need a license). Desktop version of LC.exe won't be able to execute .NET Core code, so yes, LC.exe needs to be ported if you don't want to replace the whole licensing mechanism with something different. Once its clear who will own LC.exe I have some follow-up issues to create for future improvements, historically LC.exe is very fragile and errors are hard to diagnose, but lets first get things actually working. Since WebForms are gone WinForms is probably the only one still using this kind of licensing, but I'm not entirely sure of that. |
Any news for who is going to be responsible for porting LC.exe? There's dotnet/winforms#1462 but it has seen no action since this issue was created. If you want to get this into 3.1 this probably needs to be picked up soonish. |
The issue in winforms is the right one to track that. |
My understanding is that we are not bringing LC.ee to .NET Core and instead @terrajobst will work on a newer way of licensing .NET Core apps. |
….2 (#3631) - Microsoft.DotNet.Cli.Runtime - 3.1.100-rtm.19568.2
@terrajobst @OliaG where can curious people follow this progress? |
Visual Studio Version:
Summary:
The Visual Studio support for Windows Forms licensing was implemented in dotnet/project-system#5468, and works well if you opt out of the globs. VS will automatically add it to EmbeddedResource:
However with globs turned on, we don't add a specific item if a glob covers it. Based on that, the SDK needs to cover licenses.licx in EmbeddedResource item.
Steps to Reproduce:
Expected Behavior:
Properties\licenses.licx gets created and ends up in
EmbeddedResource
itemActual Behavior:
Properties\licenses.licx gets created but ends up in
None
item typeUser Impact:
Licensed controls don't work at runtime.
The text was updated successfully, but these errors were encountered: