Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Replace RSS generator (node-rss) with dylang RSS generator (rss) #258

Closed
julianlam opened this issue Sep 3, 2013 · 4 comments
Closed

Replace RSS generator (node-rss) with dylang RSS generator (rss) #258

julianlam opened this issue Sep 3, 2013 · 4 comments
Assignees
Milestone

Comments

@julianlam
Copy link
Member

We are currently using the package node-rss as our RSS generator, but this package relies on the libxml package to output XML, which requires a lengthy compile step that continually causes npm to exit with a non-zero status code. Because it actually compiles the libxml library, it also means I cannot simply copy/paste the node_modules folder into different directories, as it the libxml library errors out.

Long story short, it's impossible to use, nobody upstream has any inclination to fix it, so I think we should switch to the rss package instead (more stars, more commits, more forks).

@ghost ghost assigned psychobunny Sep 3, 2013
@julianlam
Copy link
Member Author

In fact, it seems even easier to generate an RSS feed using this, than node-rss:

https://github.com/dylang/node-rss

@psychobunny
Copy link
Contributor

I actually used dylang's RSS module before and had an
overall bad experience with it. You're welcome to switch back the code is there in a previous commit maybe around May ish (on mobile will get commit hash later)

iirc you weren't able to load the feed in chrome.

@julianlam
Copy link
Member Author

Alright thanks. I'll give it a try tonight.
On Sep 3, 2013 8:23 PM, "psychobunny" notifications@github.com wrote:

I actually used dylang's RSS module before and had an
overall bad experience with it. You're welcome to switch back the code is
there in a previous commit maybe around May ish (on mobile will get commit
hash later)

iirc you weren't able to load the feed in chrome.


Reply to this email directly or view it on GitHubhttps://github.com//issues/258#issuecomment-23757801
.

@ghost ghost assigned julianlam Sep 4, 2013
@julianlam
Copy link
Member Author

Closed via bcbcf40

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

No branches or pull requests

2 participants