Skip to content

Commit

Permalink
Fix issue #248 where excluding pdb files from publishing broke SAM de…
Browse files Browse the repository at this point in the history
…bugging
  • Loading branch information
normj committed Nov 8, 2022
1 parent 599a350 commit 6fedbab
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/Amazon.Lambda.Tools/Amazon.Lambda.Tools.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
<Copyright>Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.</Copyright>
<Product>AWS Lambda Tools for .NET CLI</Product>
<TreatWarningsAsErrors>false</TreatWarningsAsErrors>
<Version>5.6.0</Version>
<Version>5.6.1</Version>
</PropertyGroup>
<ItemGroup>
<Content Include="Resources\build-lambda-zip.exe">
Expand Down
3 changes: 2 additions & 1 deletion src/Amazon.Lambda.Tools/LambdaPackager.cs
Original file line number Diff line number Diff line change
Expand Up @@ -589,7 +589,8 @@ private static IDictionary<string, string> GetFilesToIncludeInArchive(string pub
if (flattenRuntime && relativePath.StartsWith(RUNTIME_FOLDER_PREFIX))
continue;

if (relativePath.EndsWith(".dbg", StringComparison.OrdinalIgnoreCase) || relativePath.EndsWith(".pdb", StringComparison.OrdinalIgnoreCase)) // Don't include debugging symbols
// Native debug symbols are very large and are being excluded to keep deployment size down.
if (relativePath.EndsWith(".dbg", StringComparison.OrdinalIgnoreCase))
continue;

includedFiles[relativePath] = file;
Expand Down

0 comments on commit 6fedbab

Please sign in to comment.