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

 
 

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

Submitted by:  Philip Nienhuis <philipnienhuis>
Submitted on:  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:  Ready For Test Assigned to:  None
Originator Name:  Philip Nienhuis Open/Closed:  Open
Release:  dev Operating System:  Microsoft Windows

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

( 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>
Project 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>
Project Administrator
Wed 10 Oct 2018 05:57:20 PM UTC, comment #9:

How about something like this (attached)?

(file #45178)

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

Thanks, retitled.

Mike Miller <mtmiller>
Project Administrator
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>
Project 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>
Project Administrator
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>
Project 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>
Project Member
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>
Project 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>
Project 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>
Project 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>
Project 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 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)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follow 9 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    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.3