Skip to content
This repository was archived by the owner on Nov 1, 2023. It is now read-only.

Conversation

@Porges
Copy link
Member

@Porges Porges commented Jun 28, 2023

Make them a bit smaller by packaging into single files and enabling invariant-globalization mode.

@codecov-commenter
Copy link

codecov-commenter commented Jun 29, 2023

Codecov Report

Merging #3248 (1e9b66b) into main (0bac434) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3248   +/-   ##
=======================================
  Coverage   31.45%   31.45%           
=======================================
  Files         307      307           
  Lines       37202    37202           
=======================================
  Hits        11703    11703           
  Misses      25499    25499           

@Porges Porges changed the title Trim LibFuzzerDotNetLoader Update packaging of LibFuzzerDotNetLoader Jun 29, 2023
@Porges Porges changed the title Update packaging of LibFuzzerDotNetLoader Update packaging of LibFuzzerDotNetLoader & SharpFuzz Jun 29, 2023
@Porges Porges marked this pull request as ready for review June 29, 2023 01:49
@Porges Porges changed the title Update packaging of LibFuzzerDotNetLoader & SharpFuzz Update packaging of LibFuzzerDotNetLoader Jun 29, 2023
@Porges Porges enabled auto-merge (squash) July 18, 2023 04:05
@Porges Porges merged commit 25f1257 into main Jul 18, 2023
Porges added a commit that referenced this pull request Jul 24, 2023
@Porges Porges mentioned this pull request Jul 24, 2023
Porges added a commit that referenced this pull request Jul 24, 2023
This reverts commit 25f1257.

The change broke the libfuzzer dotnet template (I could have sworn I validated first with check-pr but…)

Two things didn't work:
1. the path to the LibFuzzerDotNetLoader needed to change; I attempted to fix this in #3324 
2. minidumps weren't being captured, for an unknown reason. I haven't figured out why yet, thus I'm going to simply revert the change so we have this working for the next release.
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Jul 26, 2023
@Porges Porges deleted the trim-libfuzzerdotnetloader branch September 7, 2023 23:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants