bufferbloat-net icon indicating copy to clipboard operation
bufferbloat-net copied to clipboard

Hugo has a new release

Open dtaht opened this issue 8 years ago • 14 comments

and complains a lot when generating the site.

dtaht avatar Feb 10 '17 18:02 dtaht

Hmmm... No errors on Hugo version 0.16, but I see that version 0.18 came out in December.

Are we being pushed to upgrade?

richb-hanover avatar Feb 10 '17 18:02 richb-hanover

Yeah, Hugo > 0.16 are won't build the site. See https://github.com/spf13/hugo/issues/2891

tohojo avatar Feb 10 '17 19:02 tohojo

Hah. It may break - but it breaks faster than ever - rendering the whole site in 1036ms on my intel box, and 3ms or so on my pi2.

dtaht avatar Feb 10 '17 21:02 dtaht

1036 msec? I bet you mean 1036 usec...

Also - I updated Toke's ticket in the Hugo issue tracker: https://github.com/spf13/hugo/issues/3026

richb-hanover avatar Feb 10 '17 21:02 richb-hanover

usec, yes. hugo is amazing. I would be watching ikiwiki still processing its first file at that point....

dtaht avatar Feb 10 '17 21:02 dtaht

Well, opening another issue did not help, exactly... :/

tohojo avatar Feb 11 '17 11:02 tohojo

I don't blame bep for being testy. But how do we un-break things?

dtaht avatar Feb 12 '17 00:02 dtaht

I thought Toke's response to his original ticket #2891 was appropriate. I also have the sense that bep feels a little defensive. The changelogs go to great lengths to say how happy everyone will be with the "everything's a page" model of 0.18, making me think that he got a lot of pushback.

Depending on how grumpy I (still) feel, I may update my forum posting to ask if anyone is planning to mark those commands as unsupported in the documentation.

But... Dave's note is on point - how do we get unstuck?

Is it as simple as:

  1. Changing every .md in a link to "/", e.g., s/.md)/\//
  2. Adding the proper path prefix to each relative link, that is, for all files in /cerowrt/wiki, prepend /cerowrt/wiki/ to the link, if it's not a full path e.g.,

[FAQ](FAQ.md) => [FAQ](/cerowrt/wiki/FAQ/) but [Other](/bloat/wiki/Other.md) => [Other](/bloat/wiki/Other/) ?

If that's all we have to do, we only need to substitute the proper path prefix for four prefixes: cerowrt, bloat, make-wifi-fast, codel. That's not insurmountable...

richb-hanover avatar Feb 12 '17 01:02 richb-hanover

It's not that I mind them dropping the feature. But a note beforehand saying "there are not resources to fix this, but we'll accept a patch" would have been nice. Stonewalling and then dropping the feature is just rude.

Now, as far as fixing it: The link changes you suggest (while turning off the parsing of relative links) would turn all the links into regular relative HTML links in the output. There are two issues with this: One, we'll have to include the site prefix (/projects), which means that if that ever changes, we'll have to do another manual update of all links on the site. And I'm not sure if it works on the local site in development mode.

But more importantly, it means that Hugo will not know that the links are links to other pages; so broken links will just turn up as 404s on the live site. To get the link checking, we'll have to use one of the template functions to generate links (see https://gohugo.io/templates/functions/#urls); and they make the source be quite a bit less Wiki-like, which is bad for usability.

So going back to my previous point: Why, exactly, do we need to upgrade?

tohojo avatar Feb 12 '17 15:02 tohojo

Stonewalling and then dropping the feature is just rude.

And not updating the documentation to state that the feature is no longer operative is, too. But let's cut bep some slack, since I think we were just "one more in a series of disgruntled people" who had to make big changes to their site.

All that said, I have no need to upgrade. What's there just works. (I need to figure out how to downgrade my hugo installation on osx, but I think there's a brew incantation for that...) Can you tweak the hugo version on the server? Thanks.

richb-hanover avatar Feb 12 '17 15:02 richb-hanover

You mean the CI server? Yeah, that broke, I forget why... not sure if that can be fixed. But the server that produces the live version still runs 0.16...

tohojo avatar Feb 12 '17 16:02 tohojo

Ah yes, there's some dependency issue. The Go ecosystem does not have a concept of freezing versions. So I think I'll just nuke the CI stuff and rely on people testing on their own box before pushing ;)

tohojo avatar Feb 12 '17 16:02 tohojo

OK with me (to nuke the CI stuff, or at least the Hugo checks). It's nice that something checks to see if the merge can happen automatically. (Or perhaps that's a github feature...)

richb-hanover avatar Feb 12 '17 16:02 richb-hanover

The check for conflicts is a github feature. The CI stuff just checked if the site compiles...

tohojo avatar Feb 12 '17 17:02 tohojo