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
Tweak templates to work with all specified feeds #8
Conversation
For some reason I needed to do this to get it to work properly.
1291468
to
bba992b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quick self-review to call out some things.
mkdir -p ${build_dir}/${PLANET} | ||
cp -r ${build_dir}/css-v2 ${build_dir}/${PLANET}/css-v2 | ||
cp -r ${build_dir}/images-v2 ${build_dir}/${PLANET}/images-v2 | ||
pluto -c ../ build ${PLANET}.ini -o ${build_dir}/${PLANET} -d ${build_dir}/${PLANET} -t planet | ||
pluto -c ../ build ${PLANET//-/}.ini -o ${build_dir}/${PLANET} -d ${build_dir}/${PLANET} -t planet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems to be that the pluto
tool doesn't like the hyphen in summer-coding
, so I removed it from the ini file name and added this stripping here, and it seems to work fine now.
pluto/build/design.ini
Outdated
# Origin: /home/fedora/nicubunu/.planet.design | ||
[http://nicubunu.blogspot.com/feeds/posts/default?alt=rss] | ||
name=Nicu Buculei | ||
feed=http://nicubunu.blogspot.com/feeds/posts/default?alt=rss |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a lot of non-HTTPS links in these entries. We should think about whether we should change those to HTTPS where possible.
pluto/build/edited.ini
Outdated
; link = http://morefedora.blogspot.com | ||
; feed = http://morefedora.blogspot.com/feeds/posts/default/-/justfedora/?alt=rss | ||
; avatar = http://www.gravatar.com/avatar/a47795df6968fe5645cd3b6b94235e86?s=80 | ||
; ; author = fasname |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should think about which fields are optional and which are required. This will have an impact on where conditionals are needed in the template files.
No description provided.