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

Changed submitted file format to DocBook #105

Closed
wants to merge 16 commits into from
Closed

Conversation

LTennis
Copy link

@LTennis LTennis commented Aug 8, 2022

Do you need/want the jpeg?

This is the proposed update to the LDP rp_filter documentation with the jpeg and source of the graphic embedded in the document
Converted OpenOffice format to DocBook.  Hopefully that's what you want, if not let me know what i need to do.
@ser
Copy link
Contributor

ser commented Aug 8, 2022

Hi, attaching graphics is described here I think:

https://tldp.org/LDP/LDP-Author-Guide/html/inserting-pictures.htm

Please have a look into similar howtos which are attaching graphics as examples to be sure how does it work.

Alternate format as requested by the documentation.
Fixed deficiencies.
@LTennis
Copy link
Author

LTennis commented Aug 8, 2022 via email

@ser
Copy link
Contributor

ser commented Aug 9, 2022

You definitely need to edit xml file as without it your graphics is not referenced - this is why I suggested to have a look into other documents as examples. EPS is not required, JPG or PNG is completely enough.

Modified to use TLDP method of including a picture
@LTennis
Copy link
Author

LTennis commented Aug 9, 2022 via email

@ser
Copy link
Contributor

ser commented Aug 9, 2022

Hi, you correctly noticed that situation is not correct. You need to edit the source file in its very location, modifying it's content. At the moment you just added a new file to our repository, which has no reference at all. Try to think it out, if any problems, just comment here.

@LTennis
Copy link
Author

LTennis commented Aug 9, 2022 via email

@ser
Copy link
Contributor

ser commented Aug 10, 2022

Hi, is this the Howto which are you updating?

https://github.com/tLDP/LDP/blob/master/LDP/howto/docbook/Adv-Routing-HOWTO.sgml

@LTennis
Copy link
Author

LTennis commented Aug 10, 2022 via email

@ser
Copy link
Contributor

ser commented Aug 11, 2022

OK, so instead of creating a new file, you need to modify existing one you just pointed on :) And please put the assisting graphics in the same subdirectory.

@LTennis
Copy link
Author

LTennis commented Aug 11, 2022 via email

@ser
Copy link
Contributor

ser commented Aug 12, 2022

Sure, I can understand, this is why I'm guiding you to reach the goal :-)

You can edit file using a web browser, or using a "git" command on your PC.

Editing on the web is simple if you need to just edit one file:

https://docs.github.com/en/repositories/working-with-files/managing-files/editing-files

@ser
Copy link
Contributor

ser commented Aug 12, 2022

@ser
Copy link
Contributor

ser commented Aug 13, 2022

We are getting closer, but you can't add these tags far inside the document:

<?xml version="1.0" encoding="UTF-8"?>

@LTennis
Copy link
Author

LTennis commented Aug 13, 2022 via email

@ser
Copy link
Contributor

ser commented Aug 13, 2022

Not really, I included example only of unwanted syntax, there is more, eg.

<!DOCTYPE article PUBLIC "-//OASIS//DTD DocBook XML V4.1.2//EN" "http://www.oasis-open.org/docbook/xml/4.1.2/docbookx.dtd">
<article lang="">

Please check syntax of the whole file with a proper lint tool, for example:

https://github.com/davidmalcolm/docbook-lint

or any other tool you find suitable for you.

@LTennis
Copy link
Author

LTennis commented Aug 15, 2022 via email

@LTennis
Copy link
Author

LTennis commented Aug 15, 2022 via email

@ser
Copy link
Contributor

ser commented Aug 16, 2022

Hi, sorry for delays with reply about lint - I'm struggling with my work tasks and I need some time to give you valuable answers. Give me some more time please.

About the editor: I would recommend to write this question to discuss@en.tldp.org list - there are smarter guys sitting there than me.

@LTennis
Copy link
Author

LTennis commented Aug 16, 2022 via email

@ser
Copy link
Contributor

ser commented Aug 16, 2022

I've just found this friendly guide in my links:

https://opensource.com/article/17/9/docbook

And it points to the linter you should have ready available in your Linux, xmllint, in my ubuntu it is in package: libxml2-utils

@LTennis
Copy link
Author

LTennis commented Aug 19, 2022 via email

@ser
Copy link
Contributor

ser commented Aug 20, 2022

I would recommend you to briefly browse the nearby howtos and check how they are written - you can compare the results with the sources and learn by example.

Unfortunately we had a general power failure in the serverhouse and admins still did not recover our machines - so the website is currently offline, but I am sure you can find copies of our documents somewhere meanwhile. On Monday we are 100% back online. Sorry :)

@pbiering
Copy link
Contributor

Related to changes of Adv-Routing-HOWTO.sgml: makes this sense to insert XML into an SGML code?

@LTennis
Copy link
Author

LTennis commented Oct 11, 2022 via email

@pbiering
Copy link
Contributor

I did the edit using a web browser, hopefully i did it correctly.

No, you have to run the SGML through a validitator first and usually SGML is generated from e.g. LyX or other editors, which can export SGML.

Direct editing of SGML is dangerous.

@LTennis
Copy link
Author

LTennis commented Oct 15, 2022 via email

@pbiering
Copy link
Contributor

pulled your master from https://raw.githubusercontent.com/LTennis/LTennis-LDP/master/LDP/howto/docbook/Adv-Routing-HOWTO.sgml and ran validator:

LC_ALL=C openjade Adv-Routing-HOWTO.sgml
openjade:Adv-Routing-HOWTO.sgml:5103:2:E: "DOCTYPE" declaration not allowed in instance
openjade:Adv-Routing-HOWTO.sgml:5105:16:E: document type does not allow element "ARTICLE" here
...

-> You've implanted a complete SGML doc into an existing SGML incl. header...this is not passing the validator.

Please adjust your PR that it is passing the validator.

Removed the xml, DOCTYPE and article references.
@LTennis
Copy link
Author

LTennis commented Oct 17, 2022 via email

pbiering added a commit to pbiering/LDP that referenced this pull request Oct 18, 2022
pbiering added a commit to pbiering/LDP that referenced this pull request Oct 18, 2022
pbiering added a commit to pbiering/LDP that referenced this pull request Oct 18, 2022
@pbiering
Copy link
Contributor

unfortunatly it's not working in this way, had to create a new PR with adjusted sgml (replacing unsupported chars like opening-", closing-", long-dash, Unicode c2a0) also relocated to a new subdirectory to align storage path of pictures with other SGML docs:

#106

Please submit next time only changes direct to SGML, thank you!

@pbiering pbiering closed this Oct 18, 2022
@LTennis
Copy link
Author

LTennis commented Oct 20, 2022 via email

@pbiering
Copy link
Contributor

Based on your original https://raw.githubusercontent.com/LTennis/LTennis-LDP/master/LDP/howto/docbook/Adv-Routing-HOWTO.sgml following problematic chars found:

# individual dash "opening":
openjade:Adv-Routing-HOWTO.sgml:5198:238:E: non SGML character number 128
openjade:Adv-Routing-HOWTO.sgml:5198:239:E: non SGML character number 156

# individual dash "closing":
openjade:Adv-Routing-HOWTO.sgml:5198:270:E: non SGML character number 157

# long dash:
openjade:Adv-Routing-HOWTO.sgml:5233:209:E: non SGML character number 147

also it didn't like <para/> and <listitem/> (simply removed) and the <graphic ...></graphic> statement (whole section replaced by <mediaobject>...</mediaobject>

Can't replay the unicode c2a0 issue, potentially caused by local tests.

PDF submission is not a good idea, because SGML is used to create various formats like TXT, single page HTML, multipage HTML and tLDP-styled PDF.

Here is a list of SGML editors: https://www.ibiblio.org/gferg/ldp/sgml_edit.html

But for further extensions is probably simpler, editing the SGML directly e.g in vim which supports syntax highlighting.

You can verify (after installing locally openjade) the SGML all the time by creating a single-page HTML:

openjade -t sgml -i html -V nochunks -d "<path/to>/LDP/LDP/builder/dsssl/ldp.dsl#html"  Adv-Routing-HOWTO.sgml >Adv-Routing-HOWTO.html

@LTennis
Copy link
Author

LTennis commented Nov 3, 2022 via email

@pbiering
Copy link
Contributor

pbiering commented Nov 5, 2022

@LTennis : the "dsl" file is located here: https://github.com/tLDP/LDP/tree/master/LDP/builder/dsssl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants