Skip to content
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

identify .NET single file bundles #841

Closed
wants to merge 13 commits into from
Closed

identify .NET single file bundles #841

wants to merge 13 commits into from

Conversation

sara-rn
Copy link
Contributor

@sara-rn sara-rn commented Nov 9, 2023

.NET single file deployments allow all the dependencies to be included in a single binary. Other capa rules might not be relevant for this single file bundles. Perhaps a warning could be displayed, same as for MSI. Additional information can be found in https://learn.microsoft.com/en-us/dotnet/core/deploying/single-file/overview?tabs=cli

Copy link

google-cla bot commented Nov 9, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Comment on lines 16 to 17
- match: contains PDB path
- string: "singlefilehost.pdb"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you add a description why this may be a good signal?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure. this is the symbol file for the native executable when publish single file is set to True

README.md Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove this file from the PR

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, understood, didn't know how to I guess, can you reject this specific commit?

Comment on lines 3 to 4
name: bundled with .NET single-file deployment
namespace: runtime/dotnet
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this go into the internal\limitation\file directory/namespace to avoid further processing?

README.md Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, I meant from the changes, not to delete the file :)

@mr-tz
Copy link
Collaborator

mr-tz commented Nov 15, 2023

Cool, can you address the parsing/linter fail and then this should be good to go.

@mr-tz
Copy link
Collaborator

mr-tz commented Nov 17, 2023

There's also packaged as single-file .NET application which we should consider as part of this update.

@sara-rn sara-rn closed this Nov 17, 2023
@sara-rn
Copy link
Contributor Author

sara-rn commented Nov 17, 2023

will create a new PR for this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants