“Build Warning: Layout 'nil' requested in atom.xml does not exist.” #2712

Closed
bdesham opened this Issue Aug 8, 2014 · 4 comments

Comments

Projects
None yet
5 participants
@bdesham

bdesham commented Aug 8, 2014

I want most of the pages in the root of my site to use the “new” layout, so I have this in my _config.yml:

defaults:
    -
        scope:
            path: ""
            type: page
        values:
            layout: new

Since “new” is an HTML template, though, I obviously don’t want it to be used for e.g. my Atom feed. I included this line in the front matter of atom.xml:

layout: nil

This worked fine in Jekyll 2.0.3. I’m now using Jekyll 2.2.0, and while atom.xml seems to be generated correctly, I get errors like these when building the site:

Build Warning: Layout 'nil' requested in atom.xml does not exist.
Build Warning: Layout 'nil' requested in index.html does not exist.

Is specifying a layout of nil not the correct way to indicate that I don’t want a layout to be used for a page? If it is the correct way, errors like this should not be shown.

@kleinfreund

This comment has been minimized.

Show comment
Hide comment
@kleinfreund

kleinfreund Aug 8, 2014

Contributor

After introducing a warning for a missing/non-existent layout, we learned that nil is not the correct way to do it. Use layout: null instead.

Contributor

kleinfreund commented Aug 8, 2014

After introducing a warning for a missing/non-existent layout, we learned that nil is not the correct way to do it. Use layout: null instead.

@bdesham

This comment has been minimized.

Show comment
Hide comment
@bdesham

bdesham Aug 8, 2014

OK, that fixed it, thanks.

bdesham commented Aug 8, 2014

OK, that fixed it, thanks.

@bdesham bdesham closed this Aug 8, 2014

tkawachi added a commit to tkawachi/tkawachi.github.com that referenced this issue Aug 20, 2014

jbytheway added a commit to jbytheway/poole that referenced this issue Aug 21, 2014

@jasonkliu jasonkliu referenced this issue in jasonkliu/demos Aug 21, 2014

Closed

Layout warning #1

jbytheway added a commit to jbytheway/jbytheway.github.io that referenced this issue Aug 22, 2014

@ridwanahmedkhan

This comment has been minimized.

Show comment
Hide comment
@ridwanahmedkhan

ridwanahmedkhan Aug 23, 2014

thanks @kleinfreund for the solution.

thanks @kleinfreund for the solution.

cbeams added a commit to cbeams/chris.beams.io that referenced this issue Aug 23, 2014

@dkarter

This comment has been minimized.

Show comment
Hide comment
@dkarter

dkarter Aug 28, 2014

thanks @kleinfreund that worked!

dkarter commented Aug 28, 2014

thanks @kleinfreund that worked!

@barryclark barryclark referenced this issue in barryclark/jekyll-now Sep 14, 2014

Merged

Update feed.xml Layout declaration #23

itorres added a commit to itorres/itorres.net that referenced this issue Oct 11, 2014

davidcelis added a commit to davidcelis/davidcelis.github.io that referenced this issue Oct 22, 2014

Change the layout in feed.xml to null
This will squelch a warning (see jekyll/jekyll#2712)

Signed-off-by: David Celis <me@davidcel.is>

@quiver quiver referenced this issue in turnbullpress/james_blog Oct 25, 2014

Closed

suppress build warnings of jekyll 2.3 #1

@pathawks pathawks referenced this issue in SeanKilleen/seankilleen.github.io Nov 3, 2014

Merged

Use `layout: null` instead of `layout: nil` #34

spier added a commit to spier/spier.github.io that referenced this issue Nov 18, 2014

qiwihui added a commit to qiwihui/qiwihui that referenced this issue Nov 22, 2014

@mchelen mchelen referenced this issue in okfn/okfn.github.com Dec 8, 2014

Merged

use "null" layout to avoid jekyll warnings #251

cpjobling added a commit to cpjobling/jekyll-bootstrap that referenced this issue Dec 11, 2014

ttmmghmm pushed a commit to ttmmghmm/ttmmghmm.github.io that referenced this issue Dec 28, 2014

@samervin samervin referenced this issue in scotte/jekyll-clean Dec 31, 2014

Merged

Change 'none' to 'null' to avoid jekyll build error #1

Andersos pushed a commit to Andersos/andersos.net that referenced this issue Jan 1, 2015

holman added a commit to holman/left that referenced this issue Jan 1, 2015

Merge pull request #33 from Andersos/patch-1
Change 'nil' to 'null' to avoid error see jekyll/jekyll#2712

syelle added a commit to syelle/syelle.github.com that referenced this issue Jan 5, 2015

Compatibility for Jekyll update
Jekyll stopped using layout:nil to indicate that a template should ignore a default layout. It now looks for 'null' instead.

Details: jekyll/jekyll#2712

mingt added a commit to mingt/greyshade that referenced this issue Jan 13, 2015

katyhuff added a commit to katyhuff/berkeley that referenced this issue Jan 27, 2015

jamchamb added a commit to jamchamb/Thinny that referenced this issue Feb 5, 2015

brunelli pushed a commit to camporez/Thinny that referenced this issue Feb 6, 2015

Kevin-M-Smith added a commit to Kevin-M-Smith/jekyll-clean that referenced this issue Mar 12, 2015

Set feed.xml layout to 'null' instead of 'none'
Using 'none' as the layout type in feed.xml results in a build warning:

```
Build Warning: Layout 'none' requested in feed.xml does not exist.
```

Using 'null' in place of 'none' resolves the issue. 

References:
See this [[issue]](jekyll/jekyll#2712 (comment)).

jc00ke added a commit to jc00ke/jc00ke.github.io that referenced this issue Mar 22, 2015

bhardin pushed a commit to bhardin/bhardin.github.com that referenced this issue Mar 25, 2015

Brett Hardin
Updating to Jekyll 2.4.0
layout nil has been replaced with layout null.

jekyll/jekyll#2712

JRaspass added a commit to cv-library/cv-library.github.io that referenced this issue Mar 27, 2015

christianvuerings added a commit to denbuzze/denbuzze.github.com that referenced this issue Apr 4, 2015

spinningarrow added a commit to spinningarrow/pdxgit.github.com that referenced this issue Apr 6, 2015

Use `layout: null` instead of `layout: nil`
Apparently this is the right way to do it (jekyll/jekyll#2712) and Jekyll shows a warning if it is set to `nil`.

@spinningarrow spinningarrow referenced this issue in pdxgit/pdxgit.github.com Apr 6, 2015

Merged

Use `layout: null` instead of `layout: nil` #19

mattdsteele added a commit to mattdsteele/steele.blue that referenced this issue Apr 17, 2015

adangtran87 added a commit to adangtran87/adangtran87.github.io that referenced this issue Apr 25, 2015

yxjxx pushed a commit to yxjxx/yxjxx.github.io that referenced this issue Aug 18, 2015

@ashfurrow ashfurrow referenced this issue in artsy/artsy.github.io Aug 20, 2015

Merged

@orta: Specifies no layout for category feeds. #137

carlosmaya pushed a commit to mxabierto/dgm that referenced this issue Dec 11, 2015

dideler added a commit to dideler/dideler.github.com that referenced this issue Jan 17, 2016

bcclaywell added a commit to fredhutchio/fredhutch.io that referenced this issue Feb 3, 2016

@asl97 asl97 referenced this issue in erjjones/erjjones.github.com Feb 23, 2016

Merged

Fix build warning in atom.xml #4

overthink added a commit to overthink/overthink.github.io that referenced this issue Jul 12, 2016

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.