Skip to content
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

can't el-get install since 0.16 broke node links in flycheck.texi #299

Closed
ronaldevers opened this issue Jan 13, 2014 · 4 comments
Closed

can't el-get install since 0.16 broke node links in flycheck.texi #299

ronaldevers opened this issue Jan 13, 2014 · 4 comments
Assignees
Labels

Comments

@ronaldevers
Copy link

@ronaldevers ronaldevers commented Jan 13, 2014

Hey, you appear to have accidentally broken the docs in release 0.16, specifically in the release commit 53ccbf6. el-get tries to run makeinfo flycheck.info in the doc folder but make info complains about the node next/prev/up pointers. I tried to fix it myself to send a pull request but failed because I couldn't figure out what you want pointing to what. Here is the error: (the warning was already there in older versions and can be ignored I think)

[ronald@thing] ~/.emacs.d/el-get/flycheck/doc [master] # makeinfo flycheck.texi
flycheck.texi:5: warning: unrecognized encoding name `UTF-8'.
/Users/ronald/.emacs.d/el-get/flycheck/doc//flycheck.texi:3299: Next field of node `master' not pointed to (perhaps incorrect sectioning?).
/Users/ronald/.emacs.d/el-get/flycheck/doc//flycheck.texi:3303: This node (Credits) has the bad Prev.
/Users/ronald/.emacs.d/el-get/flycheck/doc//flycheck.texi:3183: Prev field of node `0.16' not pointed to.
/Users/ronald/.emacs.d/el-get/flycheck/doc//flycheck.texi:3037: This node (0.15) has the bad Next.
/Users/ronald/.emacs.d/el-get/flycheck/doc//flycheck.texi:3037: Next field of node `0.15' not pointed to (perhaps incorrect sectioning?).
/Users/ronald/.emacs.d/el-get/flycheck/doc//flycheck.texi:3299: This node (master) has the bad Prev.
makeinfo: Removing output file `/Users/ronald/.emacs.d/el-get/flycheck/doc/flycheck.info' due to errors; use --force to preserve.

Hope this is an easy fix that you can make, thanks in advance.

In the meantime the el-get recipe can be patched not to build docs like so:

(:name flycheck
   :build nil
   :info nil)
@ghost ghost assigned lunaryorn Jan 13, 2014
@lunaryorn lunaryorn closed this in 3cbbade Jan 13, 2014
@ronaldevers
Copy link
Author

@ronaldevers ronaldevers commented Jan 13, 2014

nice, thanks 👍

@lunaryorn
Copy link
Contributor

@lunaryorn lunaryorn commented Jan 13, 2014

@ronaldevers Please note, that I generally build the manual with Texinfo 5, while you are using Texinfo 4, and I don't support El-get.

As such, your setup might break any time without prior notice, and I probably won't bother to fix it. Notably, I'll not try to support Texinfo 4, and I won't notify El-get maintainers of changes to Flycheck which might break their recipes.

GNU ELPA (from Marmalade or MELPA) is the only supported way to install Flycheck, so please be aware, that you are using El-get at your own risk.

@ronaldevers
Copy link
Author

@ronaldevers ronaldevers commented Jan 13, 2014

Got it :)

On Mon, Jan 13, 2014 at 2:59 PM, Sebastian Wiesner <notifications@github.com

wrote:

@ronaldevers https://github.com/ronaldevers Please note, that I
generally build the manual with Texinfo 5, while you are using Texinfo 4,
and I don't support El-get.

As such, your setup might break any time without prior notice, and I
probably won't bother to fix it. Notably, I'll not try to support Texinfo
4, and I won't notify El-get maintainers of changes to Flycheck which might
break their recipes.

GNU ELPA (from Marmalade or MELPA) is the only supported way to install
Flycheck, so please be aware, that you are using El-get at your own risk
.


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

@lunaryorn
Copy link
Contributor

@lunaryorn lunaryorn commented Jan 14, 2014

@ronaldevers Great. No offence meant, just wanted to make sure you're aware of this :) Thank you for reporting this issue :)

timothyandrew added a commit to timothyandrew/dotemacs that referenced this issue Apr 7, 2015
- Not working with el-get anymore.

flycheck/flycheck#299
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.