MichaelDaum / spreadsheet-parsexlsx

parse XLSX files
http://metacpan.org/release/Spreadsheet-ParseXLSX
1 stars 3 forks source link

Failed to install Spreadsheet::ParseXLSX on the latest Strawberry Perl 5.40.0.1 #19

Closed yslee101 closed 4 weeks ago

yslee101 commented 4 weeks ago

Hello there,

Problem = Failed to install Spreadsheet::ParseXLSX on the latest Strawberry Perl 5.40.0.1 Attempt = Rolled back to Strawberry Perl 5.38.2.2, could install without problem

OS = Windows 11 CMD = cpanm Spreadsheet::ParseXLSX

Details = Failed while trying to install the dependency Crypt packages

Based on cpan -l dump, it looks like Strawberry Perl 5.38.2.2 already has Crypt and Spreadsheet::ParseXLSX pre-installed, so there's no issue when installing the packages for updates. Whereas for Strawberry Perl 5.40.0.1 those packages are not available in the installation.

Error Message = ... Checking if you have Crypt::Mode::CBC 0 ... No Checking if you have Crypt::Mode::ECB 0 ... No ... gcc -Iltm -Iltc/headers -DLTC_SOURCE -DLTC_NO_TEST -DLTC_NO_PROTOTYPES -DLTM_DESC -std=c99 -DWIN32 -DWIN64 -DPERL_TEXTMODE_SCRIPTS -DMULTIPLICITY -DPERL_IMPLICIT_SYS -DUSE_PERLIO -DUSE_MINGW_ANSI_STDIO -fwrapv -fno-strict-aliasing -mms-bitfields -Os -falign-functions -falign-jumps -falign-labels -falign-loops -freorder-blocks -freorder-blocks-algorithm=stc -freorder-blocks-and-partition -msse4.1 -maes -DARGTYPE=4 -c ltc/ciphers/aes/aes.c -o ltc/ciphers/aes/aes.o gcc -Iltm -Iltc/headers -DLTC_SOURCE -DLTC_NO_TEST -DLTC_NO_PROTOTYPES -DLTM_DESC -std=c99 -DWIN32 -DWIN64 -DPERL_TEXTMODE_SCRIPTS -DMULTIPLICITY -DPERL_IMPLICIT_SYS -DUSE_PERLIO -DUSE_MINGW_ANSI_STDIO -fwrapv -fno-strict-aliasing -mms-bitfields -Os -falign-functions -falign-jumps -falign-labels -falign-loops -freorder-blocks -freorder-blocks-algorithm=stc -freorder-blocks-and-partition -msse4.1 -maes -DARGTYPE=4 -c ltc/ciphers/aes/aes_desc.c -o ltc/ciphers/aes/aes_desc.o ltc/ciphers/aes/aes_desc.c: In function 's_aesni_is_supported': ltc/ciphers/aes/aes_desc.c:65:7: error: 'asm' undeclared (first use in this function) 65 | asm volatile ("cpuid" | ^~~ ltc/ciphers/aes/aes_desc.c:65:7: note: each undeclared identifier is reported only once for each function it appears in ltc/ciphers/aes/aes_desc.c:65:10: error: expected ';' before 'volatile' 65 | asm volatile ("cpuid" | ^~~~~ | ; gmake[1]: *** [makefile:197: ltc/ciphers/aes/aes_desc.o] Error 1 ...

MichaelDaum commented 4 weeks ago

Seems to be a compile error of Crypt::Mode::CBC, isn't it?

yslee101 commented 4 weeks ago

Hello Michael, yes it seems to be. Do you have any advice how to resolve/proceed with the package installation, if users were to use Strawberry Perl 5.40.0.1 ? Thank you very much.

MichaelDaum commented 4 weeks ago

Hm sorry, no, as I am not familiar with Strawberry Perl. Are you saying it is specific to 5.40.0.1 or rather the fact that previous Strawberries had Crypt::Mode::CBC and Spreadsheet::ParseXLSX preinstalled?

yslee101 commented 4 weeks ago

Yeah I think it is specific to the latest release 5.40.0.1

Among the lines dumped by cpan -l for Strawberry Perl 5.38.2.2 = Crypt::Mode::CBC 0.080 Crypt::Mode::ECB 0.080 Spreadsheet::ParseXLSX 0.27

While cpan -l for Strawberry Perl 5.40.0.1 = Does not contain any of those above

I am not sure if they could be requested as part of the Strawberry Perl 5.40.0.1 installation? Or if there's anything that could be updated at cpan/etc? to avoid the compilation error? Pardon my lack of knowledge in this topic.

MichaelDaum commented 4 weeks ago

Given the security fixes in Spreadsheet::ParseXLSX, this is pretty bad. From that perspective not bundling them with Strawberry upfront is better than bundling a buggy one. Most probably there is a way to update CPAN packages in 5.38.2.2. I am using https://metacpan.org/dist/cpan-outdated/view/script/cpan-outdated for that. Not sure what to do in Strawberry as I am not familiar with that tool chain. You might need to ask people on these projects.

yslee101 commented 3 weeks ago

Understood, thank you for your time Michael :)

MichaelDaum commented 3 weeks ago

good luck