libgit2 / libgit2/libgit2sharp.nativebinaries
Parent double-dot path step in application manifest asmv1:assembly/file/@name throwing error
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- PowerShell
- Star
- 35
- Fork
- 74
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
In building from NuGet, the application manifest includes a number of asmv1:assembly/file/@name values that have a double dot, e.g.,
<file name="lib\fedora-x64\native\..\libgit2-106a5f2.so" size="1198160">
This throws an error, e.g. (via SxsTrace):
ERROR: Line 449: The value lib\alpine.3.9-x64\native\..\libgit2-106a5f2.so of attribute name in element urn:schemas-microsoft-com:asm.v1^file is invalid.
It seems that the root of this issue is https://github.com/libgit2/libgit2sharp.nativebinaries/blob/9469eab266bacdbb46aed4610b3b0e6bbfc56395/nuget.package/build/net46/LibGit2Sharp.NativeBinaries.props#L11
There the double-dot parent path is hard coded to be imprinted and negate native.
I propose the removal of ..\ in attribute TargetPath or something similar. Keeping the native subdirectory shouldn't harm anything. At any rate, ISTM the syntactical constraints of asmv1:assembly/file/@name should be respected.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với mục LibGit2Sharp.NativeBinaries.props được liên kết trong issue, đặc biệt là dòng 11 và giá trị TargetPath của nó. Build lại từ NuGet và xác minh rằng application manifest được tạo không còn chứa bước đường dẫn cha và được SxsTrace chấp nhận.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Lĩnh vực
- build-system
- Loại issue
- Lỗi
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 35/100