bugGnash - The GNU Flash player - Bugs: bug #37142, --with-sysroot path not kept

 
 

bug #37142: --with-sysroot path not kept

Submitter:  Josua Mayer <artox>
Submitted:  Sat 18 Aug 2012 05:33:40 PM UTC
   
 
Category:  build Severity:  3 - Normal
Release:  master Status:  Ready For Test
Privacy:  Public Assigned to:  rsavoye
Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 02 Sep 2012 06:52:39 PM UTC, comment #3: 

Unfortunately, the better code for OpenVG support is in branch, and not master. In the branch this works correctly. I just migrated that patch from the branch to master, committed as of #5f3dd9b.

Rob Savoye <rsavoye>
Group administrator
Fri 31 Aug 2012 01:29:03 PM UTC, comment #2: 

This might not work as it should.
"if test ! -d ${with_top_level}/usr; then"
this line does not what it should and here is why:
I have found that when --with-sysroot is given, the places to look for libraries are these: $sysroot/include;$sysroot/lib
I would have to give --with-sysroot=$realsysroot/usr to get libraries in $realsysroot/usr. Now, obviously $realsysroot/usr/usr does not exist.
When I give --with-sysroot=$realsysroot, no libs are found. When I give --with-sysroot=$realsysroot/usr, the path check is triggered and configure fails.

Josua Mayer <artox>
Sun 19 Aug 2012 02:41:56 PM UTC, comment #1: 

Fixed as of commit #225758f.

Rob Savoye <rsavoye>
Group administrator
Sat 18 Aug 2012 05:33:40 PM UTC, original submission:  

when giving configure the --with-sysroot=/some/path, my /some/path is lost. From my research, the configure script keeps a variable $incllist. To this it appends the path given as sysroot. On line 525 it starts to check if directories are valid.
[5 minutes later]
I now discovered that I had a typo in my path and it didnt eexist. Still, maybe its a bad idea to silently drop a path. should the --with-sysroot value be checked for existence earlier to report an error when that dir doesnt exist?

Josua Mayer <artox>

 

(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 rsavoye (Updated the item)
  • -email is unavailable- added by artox (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 logged-in users can vote.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-08-19 rsavoye StatusNone Ready For Test
        Assigned toNone rsavoye

    Back to the top

    Powered by Savane 3.14-f13d.
    Corresponding source code