bugGNU Octave - Bugs: bug #47817, mxe-octave: error while building...

 
 

bug #47817: mxe-octave: error while building documentation of dev Octave

Submitter:  Philip Nienhuis <philipnienhuis>
Submitted:  Sun 01 May 2016 05:53:33 PM UTC
   
 
Category:  Configuration and Build System Severity:  4 - Important
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  None
Originator Name:  philipnienhuis Open/Closed:  * Closed
Release:  * dev Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Sat 02 Jul 2016 04:15:13 PM UTC, comment #7: 

I tried yesterday as well with JOBS=4 and it worked.  Closing report.

Rik <rik5>
Group administrator
Sat 02 Jul 2016 03:44:27 PM UTC, comment #6: 

As of yesterday I've tried building without --disable-docs and with JOBS= ( > 1) and it works fine now. Thanks for fixing.

So I think this bug report can be closed. Or do we want confirmation from others?

Philip Nienhuis <philipnienhuis>
Group Member
Sat 02 Jul 2016 01:58:44 PM UTC, comment #5: 

Is this still a problem?  I modified the rule in scripts/module.mk file to use the .in file instead of the generated .m file.

John W. Eaton <jwe>
Group administrator
Sat 14 May 2016 12:32:08 AM UTC, comment #4: 

I was able to successfully build by reducing the number of JOBS to 1 in settings.mk for MXE.  There may be a race condition during the build.

Rik <rik5>
Group administrator
Fri 13 May 2016 07:51:13 PM UTC, comment #3: 

Since the reason for generating the file doesn't have anything to do with the docstring, it should be fine to extract the docstring from the .in file instead of the generated .m file.  I thought I made that change.

We already do the same kind of thing for some other generated files like the lexer and parser sources.

John W. Eaton <jwe>
Group administrator
Fri 13 May 2016 04:12:51 PM UTC, comment #2: 

The problem is that DOCSTRINGS contains documentation for the file gnuplot_binary.m while DOCSTRINGS-t does not.  This, in turn, is caused by the fact that the source directory only contains gnuplot_binary.in.  A makefile action turns the .in file into a .m file but the location of the file is now in the build directory.

It would seem that this rule in scripts/module.mk needs to be modified.


$(srcdir)/scripts/DOCSTRINGS: $(FCN_FILES) $(GEN_FCN_FILES) | scripts/$(octave-dirstamp)
        $(AM_V_GEN)rm -f scripts/DOCSTRINGS-t && \
        $(PERL) $(srcdir)/scripts/mkdoc.pl "$(srcdir)" $(FCN_FILES) -- $(GEN_FCN_FILES) > scripts/DOCSTRINGS-t && \
        $(SHELL) $(srcdir)/build-aux/move-if-change scripts/DOCSTRINGS-t $@


The variable GEN_FCN_FILES currently expands to the single file "scripts/plot/util/gnuplot_binary.m".  It needs to expand to "$(builddir)/scripts/plot/util/gnuplot_binary.m".  Or the mkdoc.pl script needs to be modified to take srcdir and builddir arguments.

At any rate, these three changesets look like they might be the proximate cause of this problem.


changeset:   21673:9631aac4b635
user:        Mike Miller <mtmiller@octave.org>
date:        Tue May 03 15:11:51 2016 -0700
summary:     build: avoid writing to srcdir unless DOCSTRINGS contents change (bug #47832)

changeset:   21664:22e7b6ad93b6
user:        John W. Eaton <jwe@octave.org>
date:        Mon May 02 18:26:50 2016 -0400
summary:     * run-octave.in: Look for DOCSTRINGS file in source tree.

changeset:   21653:09c086133228
user:        John W. Eaton <jwe@octave.org>
date:        Thu Apr 28 16:44:38 2016 -0400
summary:     simplify rules for building DOCSTRING files (bug #47745)


I'm adding Mike and jwe to the CC list for this report.


Rik <rik5>
Group administrator
Fri 13 May 2016 03:13:54 AM UTC, comment #1: 

Confirmed.  Before I get the warning about scripts/DOCSTRINGS-t I have this message:


cmp: EOF on scripts/DOCSTRINGS-t



Rik <rik5>
Group administrator
Sun 01 May 2016 05:53:33 PM UTC, original submission:  

At the moment cross-building dev-Octave in mxe-octave fails during building documentation:

:
rm -f /home/philip/devel/octdev/mxe/mxe_w64_20150316/tmp-default-octave/octave-4.1.0+/libinterp/DOCSTRINGS-t && \ ............
:
:
src/octave-build-info.cc is unchanged
libgui/src/liboctgui-build-info.cc is unchanged
cmp: /home/philip/devel/octdev/mxe/mxe_w64_20150316/tmp-default-octave/octave-4.1.0+/scripts/DOCSTRINGS-t: No such file or directory
mv: cannot stat ‘/home/philip/devel/octdev/mxe/mxe_w64_20150316/tmp-default-octave/octave-4.1.0+/scripts/DOCSTRINGS-t’: No such file or directory
Makefile:26848: recipe for target '/home/philip/devel/octdev/mxe/mxe_w64_20150316/tmp-default-octave/octave-4.1.0+/scripts/DOCSTRINGS' failed
make[3]: *** [/home/philip/devel/octdev/mxe/mxe_w64_20150316/tmp-default-octave/octave-4.1.0+/scripts/DOCSTRINGS] Error 1
make[3]: Leaving directory '/home/philip/devel/octdev/mxe/mxe_w64_20150316/tmp-default-octave/octave-4.1.0+/.build'
/home/philip/devel/octdev/mxe/mxe_w64_20150316/tmp-default-octave/octave-4.1.0+/libinterp/DOCSTRINGS is unchanged
Makefile:25462: recipe for target 'install-pdf-recursive' failed
make[2]: *** [install-pdf-recursive] Error 1
make[2]: *** Waiting for unfinished jobs....


Building with --disable-docs works fine.

Philip Nienhuis <philipnienhuis>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by jwe (Posted a comment)
  • -email is unavailable- added by rik5
  • -email is unavailable- added by rik5
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by philipnienhuis (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-07-02 rik5 StatusConfirmed Fixed
        Open/ClosedOpen Closed
    2016-07-01 rik5 Severity3 - Normal 4 - Important
    2016-05-13 rik5 Carbon-Copy- Added mtmiller
        Carbon-Copy- Added jwe
    2016-05-13 rik5 StatusNone Confirmed

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code