bugGNU Octave - Bugs: bug #54746, mxe-octave: --with-ccache should...

 
 

bug #54746: mxe-octave: --with-ccache should test whether ccache is actually available

Submitter:  Philip Nienhuis <philipnienhuis>
Submitted:  Thu 27 Sep 2018 06:41:25 PM UTC
   
 
Category:  Configuration and Build System Severity:  2 - Minor
Priority:  3 - Low Item Group:  Feature Request
Status:  Fixed Assigned to:  None
Originator Name:  Philip Nienhuis 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

Thu 11 Oct 2018 12:13:38 PM UTC, comment #11: 

Yeah it would havesince I saw the full path required in the makefiles.

I tried your changes and looks like works for me

John Donoghue <lostbard>
Group Member
Wed 10 Oct 2018 06:42:04 PM UTC, comment #10: 

If I understand correctly, this would set CCACHE to /usr/bin/ccache regardless of where the program is found?  At first, I thought to use AC_CHECK_PROG differently, but then realized the absolute path is needed in the ccache-links rule in the Makefile.  So I ended up with the following two changesets:

http://hg.octave.org/mxe-octave/rev/2ee6a02e6f28
http://hg.octave.org/mxe-octave/rev/990b5e6b1970

Does this do the right thing now?

John W. Eaton <jwe>
Group administrator
Wed 10 Oct 2018 05:57:20 PM UTC, comment #9: 

How about something like this (attached)?

(file #45178)

John Donoghue <lostbard>
Group Member
Wed 03 Oct 2018 08:12:01 PM UTC, comment #8: 

Thanks, retitled.

Mike Miller <mtmiller>
Group Member
Wed 03 Oct 2018 07:56:20 PM UTC, comment #7: 

Maybe a configure check for ccache needs to be made as per comment #1 last line.

Philip Nienhuis <philipnienhuis>
Group Member
Wed 03 Oct 2018 07:37:23 PM UTC, comment #6: 

This looks resolved to me. Does anything need fixing here?

Mike Miller <mtmiller>
Group Member
Sat 29 Sep 2018 03:51:35 PM UTC, comment #5: 

I just manually edited the Makefile in the mxe-octave dir to:
USE_CCACHE := no
and
USE_SYSTEM_OPENGL := no

(I suppose reconfiguring would make no diffference)

and then llvm (and the mesa packages) gets built fine (no errors).

Philip Nienhuis <philipnienhuis>
Group Member
Fri 28 Sep 2018 09:00:02 AM UTC, comment #4: 

Also: The ccache links aren't created until one builds the "prerequisites" target manually or one builds with "make all".

Is it possible to depend(?) on this target automatically if one configured with "--with-ccache"?

Markus Mützel <mmuetzel>
Group administrator
Thu 27 Sep 2018 08:29:33 PM UTC, comment #3: 

Didn't try yet w/o ccache configure option, I'd need to start a new mxe build for that but have no time now.
Next weekend I can try.

Philip Nienhuis <philipnienhuis>
Group Member
Thu 27 Sep 2018 08:27:41 PM UTC, comment #2: 

Yes, turns out I didn't have ccache installed on that box.

With it installed (in Linux) the llvm build goes fine. So indeed it looks like ccache might need to be checked by configure.

Philip Nienhuis <philipnienhuis>
Group Member
Thu 27 Sep 2018 06:59:16 PM UTC, comment #1: 

Does it work if you omit the --with-ccache option when configuring mxe-octave?

Does it work if you install ccache on your build system before running make for mxe-octave?

If so, then maybe we should either check for ccache when configuring mxe-octave or build it along with other build tools.

John W. Eaton <jwe>
Group administrator
Thu 27 Sep 2018 06:41:25 PM UTC, original submission:  

mxe log attached.

I've configured with:
./configure
--enable-octave=default
--enable-64
--enable-devel-tools
--enable-binary-packages
--with-ccache
--disable-system-opengl
--enable-fortran-int64

(all on one line but reproduced as separate lines for clarity)

Philip Nienhuis <philipnienhuis>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #45178:  cache.patch added by lostbard (863B - application/octet-stream)
file #45109:  llvm added by philipnienhuis (3KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Updated the item)
  • -email is unavailable- added by lostbard (Updated the item)
  • -email is unavailable- added by mmuetzel (Posted a comment)
  • -email is unavailable- added by jwe (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 11 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-10-20 rik5 StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2018-10-10 jwe StatusConfirmed Ready For Test
    2018-10-10 lostbard Attached File- Added cache.patch, #45178
    2018-10-03 mtmiller Severity3 - Normal 2 - Minor
        Priority5 - Normal 3 - Low
        Item GroupBuild Failure Feature Request
        StatusNeed Info Confirmed
        Summaryllvm fails to be cross-built mxe-octave: --with-ccache should test whether ccache is actually available
    2018-10-03 mtmiller StatusNone Need Info
    2018-09-27 philipnienhuis Attached File- Added llvm, #45109

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code