On Tue, Jan 03, 2017 at 07:01:09PM +0000, Balkin7 wrote:
> On a Windows PC this is quite easy, just checkout the openocd git repository
> with default settings. Git will convert the line endings in each text file to
> windows line endings.

Oh, I remember seeing the Git for windows installer asking about
system-wide newline settings and I thought "who the hell might be
choosing something other than refraining from any mangling" and "who
the hell decided that sane default is to do some automagic
translation"...

> On possibility to prevent this is using the .gitattributes file with something
> like:
> *.texi eol=lf

Yes, this one is worth considering.

> I do not know mutch about makeinfo and its sensivity to windows line endings,
> but the patched source code works with both line ending styles ;)

By works you mean makeinfo doesn't complain or you get fully matching
Info and PDF outputs? Do you know what is the correct way to deal with
the issue? Probably it's a bug in makeinfo, do you consider reporting
it upstream?

-- 
Be free, use free (http://www.gnu.org/philosophy/free-sw.html) software!
mailto:[email protected]


---

** [tickets:#131] Build docs fails due to makeinfo incompatibility**

**Status:** new
**Milestone:** 0.9.0
**Created:** Mon Aug 22, 2016 02:14 PM UTC by Balkin7
**Last Updated:** Tue Jan 03, 2017 07:01 PM UTC
**Owner:** nobody
**Attachments:**

- 
[doc.patch](https://sourceforge.net/p/openocd/tickets/131/attachment/doc.patch) 
(833 Bytes; application/octet-stream)


When building openocd with MSYS2, the build process fails with following 
message:
~~~
make[2]: Nothing to be done for 'install-exec-am'.
restore=: && backupdir=".am$$" && \
am__cwd=`pwd` && CDPATH="${ZSH_VERSION+.}:" && cd . && \
rm -rf $backupdir && mkdir $backupdir && \
if (makeinfo --version) >/dev/null 2>&1; then \
  for f in openocd.info openocd.info-[0-9] openocd.info-[0-9][0-9] 
openocd.i[0-9] openocd.i[0-9][0-9]; do \
    if test -f $f; then mv $f $backupdir; restore=mv; else :; fi; \
  done; \
else :; fi && \
cd "$am__cwd"; \
if makeinfo   -I . \
 -o openocd.info openocd.texi; \
then \
  rc=0; \
  CDPATH="${ZSH_VERSION+.}:" && cd .; \
else \
  rc=$?; \
  CDPATH="${ZSH_VERSION+.}:" && cd . && \
  $restore $backupdir/* `echo "./openocd.info" | sed 's|[^/]*$||'`; \
fi; \
rm -rf $backupdir; exit $rc
'penocd.texi:7986: unknown command `
'penocd.texi:7987: unknown command `
make[2]: *** [Makefile:383: openocd.info] Error 1
make[2]: Leaving directory '/D/projects/openocd-code/doc'
make[1]: *** [Makefile:639: install-am] Error 2
make[1]: Leaving directory '/D/projects/openocd-code/doc'
make: *** [Makefile:513: install-recursive] Error 1
~~~

After some search on the internet I found this bug report:
http://savannah.gnu.org/bugs/?40277

I tried the proposed solution and replaced the last @ with @@ (see attached 
patch) and the build works without errors.

My MSYS2 installation use following makeinfo version:
makeinfo --version
texi2any (GNU texinfo) 6.0


---

Sent from sourceforge.net because [email protected] is 
subscribed to https://sourceforge.net/p/openocd/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/openocd/admin/tickets/options.  Or, if this is a 
mailing list, you can unsubscribe from the mailing list.
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to