bugGNU Robots - Bugs: bug #57035, build fails with the current Guile...

 
 

bug #57035: build fails with the current Guile version 2.2

Submitter:  Mike Miller <mtmiller>
Submitted:  Thu 10 Oct 2019 05:44:38 PM UTC
   
 
Severity:  3 - Normal Status:  Fixed
Privacy:  Public Assigned to:  rozzin
Open/Closed:  Closed Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 08 Feb 2021 06:52:28 AM UTC, comment #5: 

I pushed up the `just delete the AC_CHECK_LIB call' fix.

Joshua Judson Rosen <rozzin>
Group administrator
Wed 22 Jul 2020 12:41:19 AM UTC, comment #4: 

FYI, the patch with my changes are in patch #9956.

Matt Henry <mcthenry>
Group Member
Fri 17 Jul 2020 09:26:00 PM UTC, comment #3: 

My fix for this was to remove all the bits that were dealing with Guile and replace with this which should bring in Guile 2.2 if it's available but falls back to 2.0 if 2.2 isn't available.


dnl Check for Guile.  Prefer Guile 2.2 but use 2.0 if available.
GUILE_PKG([2.2], [2.0])
GUILE_FLAGS


Matt Henry <mcthenry>
Group Member
Mon 28 Oct 2019 05:39:53 PM UTC, comment #2: 

Good one, removing the AC_CHECK_LIB works for me, and does still report an error if Guile isn't installed.

Mike Miller <mtmiller>
Mon 28 Oct 2019 03:09:24 AM UTC, comment #1: 

Ooh! Looks like Guile 2.2 was actually released just a couple of days after I started working on GNU robots--what a thing to have missed!

It looks like there's actually a much simpler fix: just remove the offending AC_CHECK_LIB call.

Apparently the GUILE_FLAGS macro that's already invoked elsewhere in configure.ac calls GUILE_PKG, which in turn checks for a viable set of Guile libs (using pkg-config); and as of Guile 2.2, GUILE_PKG defaults to accepting any of 2.2, 2.0, or 1.8 (checking for the higher versions first, of course).


Joshua Judson Rosen <rozzin>
Group administrator
Thu 10 Oct 2019 05:44:38 PM UTC, original submission:  

Hi, I am responding to your call for testers on #gnu (also mtmiller on IRC). First issue, probably a few more to come :)

I cannot build with my system's version of Guile because the configure script only looks for guile-2.0. The attached patch updates the configure script to look for guile-2.2 only, and it is enough to work for me.

If you think it would be worth it to continue building against an older version of Guile, I would be willing to put in more work to make this conditional, or loop over possible versions of Guile, etc.

Mike Miller <mtmiller>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #47644:  gnurobots-guile-2.2.patch added by mtmiller (701B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mcthenry (Posted a comment)
  • -email is unavailable- added by rozzin (Posted a comment)
  • -email is unavailable- added by mtmiller (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 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-02-08 rozzin Open/ClosedOpen Closed
    2021-02-08 rozzin StatusIn Progress Fixed
    2019-10-28 rozzin StatusNone In Progress
        Assigned toNone rozzin
    2019-10-10 mtmiller Attached File- Added gnurobots-guile-2.2.patch, #47644

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code