Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update xls2csv usage since it's called with invalid parameters #3912

Open
mc-butler opened this issue Apr 10, 2018 · 9 comments
Open

Update xls2csv usage since it's called with invalid parameters #3912

mc-butler opened this issue Apr 10, 2018 · 9 comments
Labels
area: core Issues not related to a specific subsystem prio: low Minor problem or easily worked around

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/3912
Reporter adamm (amajer@….de)

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

Note

Original attachments:

@mc-butler
Copy link
Author

Changed by adamm (amajer@….de) on Apr 10, 2018 at 11:46 UTC

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Apr 12, 2018 at 6:39 UTC (comment 1)

  • Milestone changed from Future Releases to 4.8.21

Hi Andrew, looks good to me!

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Apr 12, 2018 at 8:54 UTC (comment 2)

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

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on May 27, 2018 at 13:57 UTC (comment 3)

  • Milestone changed from 4.8.21 to 4.8.22

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Dec 29, 2018 at 14:54 UTC (comment 4)

  • Milestone changed from 4.8.22 to 4.8.23

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Jun 16, 2019 at 17:22 UTC (comment 5)

  • Milestone changed from 4.8.23 to 4.8.24

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Jul 5, 2020 at 17:32 UTC (comment 6)

  • Milestone changed from 4.8.25 to 4.8.26

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Jan 10, 2021 at 14:30 UTC (comment 7)

  • Milestone changed from 4.8.26 to 4.8.27

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Apr 8, 2021 at 6:22 UTC (comment 8)

  • Milestone changed from 4.8.27 to Future Releases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues not related to a specific subsystem prio: low Minor problem or easily worked around
Development

No branches or pull requests

1 participant