github-starred/libxlsxwriter

Fork 0

mirror of https://github.com/jmcnamara/libxlsxwriter.git synced 2026-05-15 14:15:54 -06:00

[GH-ISSUE #228] Turn on ZIP64 support #184

New issue

Closed

opened 2026-05-05 11:51:13 -06:00 by gitea-mirror · 10 comments

gitea-mirror commented

2026-05-05 11:51:13 -06:00

Owner

Originally created by @evanmiller on GitHub (May 15, 2019).
Original GitHub issue: https://github.com/jmcnamara/libxlsxwriter/issues/228

Originally assigned to: @jmcnamara on GitHub.

I get an error when creating an especially large XLSX file (see #227). The underlying error is ZIP_BADZIPFILE, triggered at

https://github.com/jmcnamara/libxlsxwriter/blob/master/third_party/minizip/zip.c#L1731

I believe this can be fixed by changing the last argument to zipOpenNewFileInZip4_64 be 1 instead of 0 here:

https://github.com/jmcnamara/libxlsxwriter/blob/master/src/packager.c#L1063-L1069

I.e. that turns on Zip64 support which permits 4GB+ uncompressed file sizes to be written.

Originally created by @evanmiller on GitHub (May 15, 2019). Original GitHub issue: https://github.com/jmcnamara/libxlsxwriter/issues/228 Originally assigned to: @jmcnamara on GitHub. I get an error when creating an especially large XLSX file (see #227). The underlying error is `ZIP_BADZIPFILE`, triggered at https://github.com/jmcnamara/libxlsxwriter/blob/master/third_party/minizip/zip.c#L1731 I believe this can be fixed by changing the last argument to `zipOpenNewFileInZip4_64` be `1` instead of `0` here: https://github.com/jmcnamara/libxlsxwriter/blob/master/src/packager.c#L1063-L1069 I.e. that turns on Zip64 support which permits 4GB+ uncompressed file sizes to be written.

gitea-mirror

2026-05-05 11:51:13 -06:00

closed this issue
added the
feature request

short term
labels

gitea-mirror commented

2026-05-05 11:51:21 -06:00

Author

Owner

@jmcnamara commented on GitHub (May 15, 2019):

Agreed. That could/should be added. It is supported in the Python version of the library: https://xlsxwriter.readthedocs.io/workbook.html#workbook-use-zip64

@jmcnamara commented on GitHub (May 15, 2019): Agreed. That could/should be added. It is supported in the Python version of the library: https://xlsxwriter.readthedocs.io/workbook.html#workbook-use-zip64

gitea-mirror commented

2026-05-05 11:51:23 -06:00

Author

Owner

@jmcnamara commented on GitHub (Jun 7, 2019):

I've added this to master.

From the docs:

/**
 * @brief Allow ZIP64 extensions when creating the xlsx file zip container.
 *
 * @param workbook Pointer to a lxw_workbook instance.
 *
 * Use ZIP64 extensions when writing the xlsx file zip container to allow
 * files greater than 4 GB.
 *
 * @code
 *     workbook_use_zip64(workbook);
 * @endcode
 */

And from a sample file generated with the ZIP64 extensions:

$ zipdetails zip64.xlsx

0000 LOCAL HEADER #1       04034B50
0004 Extract Zip Spec      2D '4.5'
0005 Extract OS            00 'MS-DOS'
0006 General Purpose Flag  0000
     [Bits 1-2]            0 'Normal Compression'
0008 Compression Method    0008 'Deflated'
000A Last Mod Time         00210000 'Tue Jan  1 00:00:00 1980'
000E CRC                   3A495D61
0012 Compressed Length     0000014F
0016 Uncompressed Length   0000048F
001A Filename Length       0013
001C Extra Length          0014
001E Filename              '[Content_Types].xml'
0031 Extra ID #0001        0001 'ZIP64'
0033   Length              0010
0035 PAYLOAD

@jmcnamara commented on GitHub (Jun 7, 2019): I've added this to master. From the docs: ```doxygen /** * @brief Allow ZIP64 extensions when creating the xlsx file zip container. * * @param workbook Pointer to a lxw_workbook instance. * * Use ZIP64 extensions when writing the xlsx file zip container to allow * files greater than 4 GB. * * @code * workbook_use_zip64(workbook); * @endcode */ ``` And from a sample file generated with the ZIP64 extensions: ``` $ zipdetails zip64.xlsx 0000 LOCAL HEADER #1 04034B50 0004 Extract Zip Spec 2D '4.5' 0005 Extract OS 00 'MS-DOS' 0006 General Purpose Flag 0000 [Bits 1-2] 0 'Normal Compression' 0008 Compression Method 0008 'Deflated' 000A Last Mod Time 00210000 'Tue Jan 1 00:00:00 1980' 000E CRC 3A495D61 0012 Compressed Length 0000014F 0016 Uncompressed Length 0000048F 001A Filename Length 0013 001C Extra Length 0014 001E Filename '[Content_Types].xml' 0031 Extra ID #0001 0001 'ZIP64' 0033 Length 0010 0035 PAYLOAD ```

gitea-mirror commented

2026-05-05 11:51:26 -06:00

Author

Owner

@evanmiller commented on GitHub (Jun 8, 2019):

Would use_zip64 make more sense as a field in lxw_workbook_options? That would seem to provide a more consistent API than a one-off function like this.

@evanmiller commented on GitHub (Jun 8, 2019): Would `use_zip64` make more sense as a field in `lxw_workbook_options`? That would seem to provide a more consistent API than a one-off function like this.

gitea-mirror commented

2026-05-05 11:51:29 -06:00

Author

Owner

@jmcnamara commented on GitHub (Jun 8, 2019):

Would use_zip64 make more sense as a field in lxw_workbook_options

That was my first thought but the Python version uses a function/method (use_zip64()) so I went with that.

The lxw_workbook_options are mainly for options that need to be defined at construction time or are very common. The zip64 option isn't in either of those categories and doesn't seem to be very commonly required. You are only the second person to request or mention it across the Perl/Python/C versions. So I'll leave it as a function for now, but I'll think about it up until the release.

@jmcnamara commented on GitHub (Jun 8, 2019): > Would `use_zip64` make more sense as a field in `lxw_workbook_options` That was my first thought but the Python version uses a function/method ([use_zip64()](https://xlsxwriter.readthedocs.io/workbook.html#workbook-use-zip64)) so I went with that. The `lxw_workbook_options` are mainly for options that need to be defined at construction time or are very common. The zip64 option isn't in either of those categories and doesn't seem to be very commonly required. You are only the second person to request or mention it across the Perl/Python/C versions. So I'll leave it as a function for now, but I'll think about it up until the release.

gitea-mirror commented

2026-05-05 11:51:33 -06:00

Author

Owner

@evanmiller commented on GitHub (Jun 8, 2019):

Well, I have been using libxlsxwriter in shipping code for a number of years, and Zip64 wasn't something I knew I needed until recently. It took some fairly deep debugging to figure out that its absence was triggering write failures. If it had been an advertised option sooner, I certainly would have enabled it. So I suspect that many clients of libxlsxwriter will choose to enable Zip64 once they are made aware of it.

One advantage of using an option instead of a function is that in the future, you will be able to make Zip64 the default (opt-out) without changing the API. The new API is opt-in only.

@evanmiller commented on GitHub (Jun 8, 2019): Well, I have been using libxlsxwriter in shipping code for a number of years, and Zip64 wasn't something I knew I needed until recently. It took some fairly deep debugging to figure out that its absence was triggering write failures. If it had been an advertised option sooner, I certainly would have enabled it. So I suspect that many clients of libxlsxwriter will choose to enable Zip64 once they are made aware of it. One advantage of using an option instead of a function is that in the future, you will be able to make Zip64 the default (opt-out) without changing the API. The new API is opt-in only.

gitea-mirror commented

2026-05-05 11:51:35 -06:00

Author

Owner

@jmcnamara commented on GitHub (Jun 8, 2019):

Okay. I'm persuaded. I'll enable it in the lxw_workbook_options.

@jmcnamara commented on GitHub (Jun 8, 2019): Okay. I'm persuaded. I'll enable it in the `lxw_workbook_options`.

gitea-mirror commented

2026-05-05 11:51:37 -06:00

Author

Owner

@jmcnamara commented on GitHub (Jun 8, 2019):

I've refactored the use_zip64 option into the constructor. It can now be called as follows:

int main() {

    lxw_workbook_options options = {.constant_memory = LXW_FALSE,
                                    .tmpdir = NULL,
                                    .use_zip64 = LXW_TRUE};

    lxw_workbook  *workbook  = workbook_new_opt("zip64.xlsx", &options);
    lxw_worksheet *worksheet = workbook_add_worksheet(workbook, NULL);

    ...

When you get a chance try it out and see what you think. The docs have been updates as well.

@jmcnamara commented on GitHub (Jun 8, 2019): I've refactored the `use_zip64` option into the constructor. It can now be called as follows: ```C int main() { lxw_workbook_options options = {.constant_memory = LXW_FALSE, .tmpdir = NULL, .use_zip64 = LXW_TRUE}; lxw_workbook *workbook = workbook_new_opt("zip64.xlsx", &options); lxw_worksheet *worksheet = workbook_add_worksheet(workbook, NULL); ... ``` When you get a chance try it out and see what you think. The docs have been updates as well.

gitea-mirror commented

2026-05-05 11:51:39 -06:00

Author

Owner

@jmcnamara commented on GitHub (Jun 8, 2019):

When you get a chance try it out and see what you think.

After I fix the failing test cases on Linux.

@jmcnamara commented on GitHub (Jun 8, 2019): > When you get a chance try it out and see what you think. After I fix the failing test cases on Linux.

gitea-mirror commented

2026-05-05 11:51:40 -06:00

Author

Owner

@jmcnamara commented on GitHub (Jun 8, 2019):

Okay. Master is fixed again.

@jmcnamara commented on GitHub (Jun 8, 2019): Okay. Master is fixed again.

gitea-mirror commented

2026-05-05 11:51:43 -06:00

Author

Owner

@evanmiller commented on GitHub (Jun 8, 2019):