Bug 7612 - zypper no longer accepts --xml argument
Summary: zypper no longer accepts --xml argument
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Server Installer (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.12.1
Assignee: Niko Lehto
URL:
Keywords: linma_tester, relnotes
Depends on:
Blocks:
 
Reported: 2020-12-16 13:51 CET by Pierre Ossman
Modified: 2021-01-21 12:47 CET (History)
2 users (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2020-12-16 13:51:21 CET
On SLES 15 you now get this running tl-setup:

> Library and binary requirements
> ===============================
> 
> ThinLinc requires a few libraries and binaries to work as expected.
> This system lacks one or more of these.
> 
> The necessary packages can be installed automatically by this wizard,
> or you can install them manually and restart the wizard.
> 
> Press Enter to continue...
> 
> Resolving packages and dependencies...The flag --xml is not known.
>  Internal error. The ThinLinc Server Setup Wizard will now terminate.
> Please send a copy of /var/log/tlsetup.log to the ThinLinc support.
> Visit https://www.cendio.com/thinlinc/support for information on how
> to contact us.

And this in the log:

> ...
> 2020-12-16 12:55:12,349:   File "/opt/thinlinc/modules/thinlinc/packageinstaller/zypperbackend.py", line 56, in __parse_file
> 2020-12-16 12:55:12,349:     return self . _parser . ParseFile ( buf )
> 2020-12-16 12:55:12,349: ExpatError: no element found: line 1, column 0

We are using "zypper --xml" but it seems that is no longer supported. Instead "zypper --xmlout" should be used. Checking "zypper --help" on SLES 12 also shows only "--xmlout" so perhaps "--xml" was never really fully supported?
Comment 3 Niko Lehto cendio 2021-01-14 16:30:44 CET
Reproduced with Nightly build 6717.

+------------+--------+
|SLES 15     | Works? |
+------------+--------+
|Without fix |  NO    |
+------------+--------+
|With fix    |  YES   |
+------------+--------+

+------------+--------+
|SLES 12     | Works? |
+------------+--------+
|Without fix |  YES   |
+------------+--------+
|With fix    |  YES   |
+------------+--------+

Also checked tlsetup.log for each of these.
Works well.
Comment 4 Linn cendio 2021-01-18 12:37:35 CET
I was able to reproduce the behaviour on both SLES 12 and 15. I tested the fix with Jenkins build 1825 and can confirm that the fix works on both versions. The output in tlsetup.log looks good, as well as the release notes. 

+------------+--------+
|SLES 15     | Works? |
+------------+--------+
|Without fix |  NO    |
+------------+--------+
|With fix    |  YES   |
+------------+--------+

+------------+--------+
|SLES 12     | Works? |
+------------+--------+
|Without fix |  YES   |
+------------+--------+
|With fix    |  YES   |
+------------+--------+

Note You need to log in before you can comment on or make changes to this bug.