bugGNU Octave - Bugs: bug #62295, installation path includes build...

 
 

bug #62295: installation path includes build path

Submitter:  Dmitri A. Sergatskov <dasergatskov>
Submitted:  Mon 11 Apr 2022 08:01:55 PM UTC
 
Category:  Configuration and Build System Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Installation Failure
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 7.1.0 Operating System:  * Any
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       

 

Wed 20 Apr 2022 08:55:10 PM UTC, comment #5: 

Marking as fixed and closing report.

Rik <rik5>
Project Administrator
Wed 13 Apr 2022 05:31:22 PM UTC, comment #4: 

Thanks for checking.

I pushed the patch to the stable branch here:
https://hg.savannah.gnu.org/hgweb/octave/rev/3c3abe6dbe58

Marking as ready for test.

Markus Mützel <mmuetzel>
Project Member
Wed 13 Apr 2022 04:16:27 PM UTC, comment #3: 

Works for me (on linux):


find ./ -name "tbcover.m"
./share/octave/7.1.1/etc/tests/fixed/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@double/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@single/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@char/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@logical/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@int8/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@int16/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@int32/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@int64/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@uint8/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@uint16/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@uint32/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@uint64/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@struct/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@cell/tbcover.m
./share/octave/7.1.1/etc/tests/fixed/@function_handle/tbcover.m

Dmitri.
--

Dmitri A. Sergatskov <dasergatskov>
Wed 13 Apr 2022 11:57:03 AM UTC, comment #2: 

I believe this would be fixed with the attached patch.

(file #53084)

Markus Mützel <mmuetzel>
Project Member
Tue 12 Apr 2022 08:29:26 PM UTC, comment #1: 

Confirmed on Linux as well. 

This is probably related to this portion of the file test/Makefile.am.

EXTRA_DIST += \
  bc-overloads-expected \
  mk-bc-overloads-tst.sh \
  mk-conv-tst.sh \
  mk-sparse-tst.sh \
  mk_bc_overloads_expected.m \
  show-failures.awk \
  $(addprefix $(srcdir)/, $(TEST_FILES)) \
  $(addprefix $(srcdir)/, $(noinst_TEST_FILES)) \
  $(MEX_TEST_SRC)

Rik <rik5>
Project Administrator
Mon 11 Apr 2022 08:01:55 PM UTC, original submission:  

As noted by Stephen Montgomery-Smith,
https://lists.gnu.org/archive/html/help-octave/2022-04/msg00005.html

some test .m files are installed in the path that includes
the original build directory path:

"""
I recently upgraded to
octave 7.1.0.  I found something rather interesting - it creates a bunch
of files whose names are:

/usr/local/bin/share/octave/7.1.0/test/fixed/xxxxxxxxxx/tbcover.m

where the xxxxxxxx is derived from the directory in which the build
takes place.  In my case, xxxxxxxx would be
usr/ports/math/octave/work/octave-7.1.0.
"""

I can confirm it with the official Windows install as well as
linux self-compile install.

E.g. on Windows I see:

dmitri@dasp53 MINGW64 /c/Program Files/GNU Octave/Octave-7.1.0
$ find ./ -name "tbcover.m"
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@cell/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@char/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@double/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@function_handle/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@int16/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@int32/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@int64/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@int8/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@logical/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@single/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@struct/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@uint16/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@uint32/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@uint64/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/@uint8/tbcover.m
./mingw64/share/octave/7.1.0/etc/tests/fixed/scratch/build/mxe-octave-w64/release/tmp-release-octave/octave-7.1.0/.build/test/tbcover.m

I guess this /tmp-release-octave/octave-7.1.0/.build/ should not be there.

Dmitri.
--

Dmitri A. Sergatskov <dasergatskov>

 

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

Attach Files:
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mmuetzel (Updated the item)
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by dasergatskov (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 project members can vote.

     

     

     

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-04-20 rik5 StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2022-04-13 mmuetzel StatusPatch Submitted Ready For Test
    2022-04-13 mmuetzel Attached File- Added bug62295-test-install-leaks-build-dirs.patch, #53084
        Item GroupNone Installation Failure
        StatusConfirmed Patch Submitted
    2022-04-12 rik5 StatusNone Confirmed

    Back to the top


    Powered by Savane 3.9