HostSystem is on ZipEntry is not respected

[macsux]

Steps to reproduce

1. Start with sample ZIP application https://github.com/icsharpcode/SharpZipLib/tree/master/samples/ICSharpCode.SharpZipLib.Samples/cs/CreateZipFile
2. Modify to set HostSystem = 3 on ZipEntry object

Also setting ExtraData is not saved as well...

Sample LinqPad script here: http://share.linqpad.net/fungmm.linq

Expected behavior

Zip file is created with HostSystem set to *unx
Need to set the OS to be able to attach file permission information properly.

Actual behavior

HostSystem flag is not respected

Version of SharpZipLib

1.1

Obtained from (only keep the relevant lines)

• Package installed using NuGet

[Numpsy]

related to / same as #325 ? (it mentions the setting of HostSystem being broken)

[macsux]

I've confirmed that the open PR fixes the issue I'm experiencing. Would be great if it can be merged a new pre-release nuget package minted, right now I've compiled myself and posted it here: https://www.myget.org/F/macsux/api/v3/index.json

[Numpsy]

Fixed by #325 / #350 ?

[macsux]

325 fixes this issue

…

On Sun., Aug. 11, 2019, 11:02 a.m. Richard Webb, ***@***.***> wrote: Fixed by #325 <#325> / #350 <#350> ? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#340?email_source=notifications&email_token=AAINFWHVMIAYGVEGT5C57MTQEASZ7A5CNFSM4HIWX6U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4BCQ5Q#issuecomment-520235126>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAINFWGAC3GSVVYGTKENCVTQEASZ7ANCNFSM4HIWX6UQ> .

[Numpsy]

The changes should be included in the new 1.2.0 release.