Page MenuHomeFreeBSD

[linux-emulation and nanobsd] some hugo/asciidoctor changes
AbandonedPublic

Authored by carlavilla on Jun 27 2020, 5:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 26, 6:39 AM
Unknown Object (File)
Feb 2 2024, 6:53 PM
Unknown Object (File)
Dec 26 2023, 12:45 PM
Unknown Object (File)
Dec 20 2023, 6:56 AM
Unknown Object (File)
Nov 9 2023, 5:58 AM
Unknown Object (File)
Oct 8 2023, 4:56 AM
Unknown Object (File)
Sep 9 2023, 4:13 PM
Unknown Object (File)
Aug 23 2023, 1:15 PM
Subscribers
None

Details

Reviewers
bcr
imp
hrs
Summary

In the linux-emulation article:

change the .... separator to ... -> this change doesn't affect the purpose of the example
Test Plan

Compile the articles

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

carlavilla created this revision.

Adding Warner for comments on nanoBSD. I see no problems with these changes personally.

imp requested changes to this revision.Jun 28 2020, 3:28 AM

All the nanobsd changes are wrong.

en_US.ISO8859-1/articles/nanobsd/article.xml
170 ↗(On Diff #73781)

The original was correct. It is an_ not a - here and below

This revision now requires changes to proceed.Jun 28 2020, 3:28 AM

@imp yes, I know that the original number have the _ character instead the - character.

But AsciiDoctor uses the _ character to put the word in italic.

But if the change it's wrong don't worry I'll find another way. Thanks.

carlavilla edited the summary of this revision. (Show Details)

NanoBSD changes deleted

@imp and @bcr can you please review the new diff again.
Thanks :)

I get this exception when I load this review in Phabricator:

Found unknown intradiff source line, expected a line beginning with "+", "-", or " " (space): \ No newline at end of file

Not sure what's going on, maybe you need to re-do the diff and upload it again?

Now the error went away... Strange, maybe a Phabricator hiccup. Anyway, this change is easy and hereby approved!

@imp and @bcr can you please review the new diff again.
Thanks :)

New stuff looks good to me.

Is there some quoting needed for NanoBSD's funky names we can do instead?

I don't know how I couldn't remember ... you just have to escape the character, instead of _ use \_

@hrs and @imp can you add your approval to the review?

Thanks for your time all of you!

Current changes are good.

This revision is now accepted and ready to land.Jun 29 2020, 2:36 PM

I don't know how I couldn't remember ... you just have to escape the character, instead of _ use \_

Cool. Are you doing that?

In D25492#563586, @imp wrote:

I don't know how I couldn't remember ... you just have to escape the character, instead of _ use \_

Cool. Are you doing that?

Yes, I'm using this and works well. This week I'm going to show all the articles converted to Hugo/AsciiDoctor and the next week I'll start with the books :)

In D25492#563586, @imp wrote:

I don't know how I couldn't remember ... you just have to escape the character, instead of _ use \_

Cool. Are you doing that?

Yes, I'm using this and works well. This week I'm going to show all the articles converted to Hugo/AsciiDoctor and the next week I'll start with the books :)

I like that very much!

hrs requested changes to this revision.Jun 29 2020, 6:17 PM

Please use … instead of three periods. It is the correct way to show a horizontal ellipsis.

This revision now requires changes to proceed.Jun 29 2020, 6:17 PM