>>> Klaus Wenninger <kwenn...@redhat.com> schrieb am 15.01.2018 um 08:41 in Nachricht <a5a3bfc9-ddfb-5cb6-797d-afb69e585...@redhat.com>: > On 01/15/2018 08:33 AM, Ulrich Windl wrote: >> >>>>> Ken Gaillot <kgail...@redhat.com> schrieb am 11.01.2018 um 17:37 in >>>>> Nachricht >> <1515688644.12807.1.ca...@redhat.com>: >>> On Thu, 2018-01-11 at 08:54 +0100, Ulrich Windl wrote: >>>> On "--crm_xml -> --xml-text": Why not simply "--xml" (XML IS text)? >>> Most Pacemaker tools that accept XML can get it from standard input ( >>> --xml-pipe), a file (--xml-file), or a literal string (--xml-text). >>> >>> Although, looking at it now, it might be nice to reduce it to one >>> option: >>> >>> --xml - standard input >>> --xml '<cib>' anything starting with '<' is literal >>> --xml file anything else is a filename >> Sounds good. Maybe some "inverted microsoft logic" could be applied: In the > past (MS-DOS times) for some parameters an argument staring with '@' meant to > read the actual parameters from the file that follows '@' (mostly because the > MS-DOS line length was limited to 127 bytes): For XML we should consider > everything as a file name, with two exceptions: >> 1) "-" stands for stdin >> 2) "@" says the rest is to be taken as file contents (not file name) >> >> So you could have XML that does not start with '<' immediately. > > What did you explicitly have in mind that wouldn't start with '<', > and that justifies introduction of a special-character like '@' that > might cause all sorts of other issues?
I don't know, but when changing the syntax anyway, who will be unable to add a '@', especially after getting "no such file or directory" errors for the XML? You may argue whether '@' is needed or desired at all, but not about complicating the syntax by requiring to add '@' in front of literal XML. Regards, Ulrich > > Regards, > Klaus > >> >> Regards, >> Ulrich >> >>> -- >>> Ken Gaillot <kgail...@redhat.com> >>> >>> _______________________________________________ >>> Users mailing list: Users@clusterlabs.org >>> http://lists.clusterlabs.org/mailman/listinfo/users >>> >>> Project Home: http://www.clusterlabs.org >>> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf >>> Bugs: http://bugs.clusterlabs.org >> >> _______________________________________________ >> Users mailing list: Users@clusterlabs.org >> http://lists.clusterlabs.org/mailman/listinfo/users >> >> Project Home: http://www.clusterlabs.org >> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf >> Bugs: http://bugs.clusterlabs.org _______________________________________________ Users mailing list: Users@clusterlabs.org http://lists.clusterlabs.org/mailman/listinfo/users Project Home: http://www.clusterlabs.org Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf Bugs: http://bugs.clusterlabs.org
