Ticket #3912 (new defect)

Opened 6 years ago

Last modified 3 years ago

Update xls2csv usage since it's called with invalid parameters

Reported by: adamm Owned by:
Priority: minor Milestone: Future Releases
Component: mc-core Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: no branch Votes for changeset:

Description

xls2csv only prints error page and returns EXIT CODE 0
when only supplied a spreadsheet without -x parameter.
-c parameter also mandatory.


Currently permitted parameters are:


usage: xls2csv -x spreadsheet.xls [-w worksheet]

[-n worksheet number] [-b charset] [-c csvfile.csv]
[-a charset] [-qshvWf]


This is xls2csv version 1.07


Which is latest version in openSUSE:Factory

Attaching git patch

Attachments

0001-Update-xls2csv-usage.patch​ (1.1 KB) - added by adamm 6 years ago.

Change History

Changed 6 years ago by adamm

comment:1 Changed 6 years ago by zaytsev

  • Milestone changed from Future Releases to 4.8.21

Hi Andrew, looks good to me!

comment:2 Changed 6 years ago by andrew_b

Are we going to drop an old version of xls2csv?
As I can see in ​man page there is incompatibility in -c option:
old: -cchar cell separator char. By default - comma.
new: -c csvfile.csv

Last edited 6 years ago by andrew_b (previous) (diff)

comment:3 Changed 6 years ago by zaytsev

  • Milestone changed from 4.8.21 to 4.8.22

comment:4 Changed 5 years ago by zaytsev

  • Milestone changed from 4.8.22 to 4.8.23

comment:5 Changed 5 years ago by zaytsev

  • Milestone changed from 4.8.23 to 4.8.24

comment:6 Changed 4 years ago by zaytsev

  • Milestone changed from 4.8.25 to 4.8.26

comment:7 Changed 3 years ago by zaytsev

  • Milestone changed from 4.8.26 to 4.8.27

comment:8 Changed 3 years ago by andrew_b

  • Milestone changed from 4.8.27 to Future Releases
Note: See TracTickets for help on using tickets.