bugGNU gv - Bugs: bug #23896, case collision with src/GV and...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #23896: case collision with src/GV and src/gv

Submitted by:  Ryan Schmidt <ryandesign>
Submitted on:  Sun 20 Jul 2008 07:32:48 AM UTC  
 
Category: Sources configurationSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Markus Steinborn <msteinbo>
Open/Closed: ClosedRelease: None

Sun 20 Jul 2008 10:06:57 AM UTC, comment #4:

Thanks for the detailed response. It it very valuable, as it is the first report that the new "setenv-code" is working (it is not needed on Linux, you see).

I am investigating for a better solution, but that may take some time. Ideas are welcome.

Markus Steinborn <msteinbo>
Project AdministratorIn charge of this item.
Sun 20 Jul 2008 08:57:48 AM UTC, comment #3:

Before reporting I did check that the issue remained with 3.6.5. I've now updated the port to that version.

http://trac.macports.org/changeset/38430

Thanks for the workaround. That will do fine for now. I went a step further and changed "GV" to another name in the Makefile, then renamed it back to "GV" after installation. I like to avoid making our users run autoreconf when possible so I modified Makefile.in instead of Makefile.am.

http://trac.macports.org/changeset/38432

Ryan Schmidt <ryandesign>
Sun 20 Jul 2008 07:56:36 AM UTC, comment #2:

Thank you for the bug report. As a quick hack, you can comment out the following lines in src/Makefile.am:

#appdefaults_DATA = GV

#GV : $(srcdir)/GV.ad
# @echo "creating GV"
# sed -e 's/^[^!]/!&/' $(srcdir)/GV.ad > $(srcdir)/GV

and run "autoreconf". Then $(srcdir)/GV is not build and installed. (It is not used anyway. It is just an template for the administrator of the system for building a system wide app-default file, so you can do a test without it.) Will this quick hack help?

Bye the way: The cited GV file on the macports-users mailing list looks like it is build with an outdated GNU gv. The actual release is version 3.6.5.

Markus Steinborn <msteinbo>
Project AdministratorIn charge of this item.
Sun 20 Jul 2008 07:35:18 AM UTC, comment #1:

Here is the thread on the macports-users mailing list about this issue:

http://lists.macosforge.org/pipermail/macports-users/2008-July/011016.html

Ryan Schmidt <ryandesign>
Sun 20 Jul 2008 07:32:48 AM UTC, original submission:

When building gv on a case-sensitive filesystem I see that, after running "make", the src directory contains both "gv" (the gv binary) and "GV" (the config file copied from GV.ad).

When building gv on a case-insensitive filesystem, of course only one of these can exist at a time. On my system (a Power Mac G4 466 MHz with Mac OS X 10.5.4) the one that survives is the gv binary. But I have a report from someone on whose system (a PowerBook G4 with Mac OS X 10.5.3) the one that survives is the GV config file. On both our systems, the file that survives gets copied to both ${prefix}/bin/gv and ${prefix}/lib/gv/GV, so in both cases, one of those files is the wrong one. In my case, the binary is installed to the place where the config file should be, and in his case, the config file gets installed to the place where the binary should be.

I recommend that the gv build process be changed so that there is not a case collision so that gv can be properly built on all systems, even those with case-insensitive filesystems (which is the default on all versions of Mac OS X so far).

Ryan Schmidt <ryandesign>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by msteinbo (Posted a comment)
  • -unavailable- added by ryandesign (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 25 Sep 2008 06:02:07 PM UTCmsteinboStatusWorkaround available=>Fixed
      Open/ClosedOpen=>Closed
    Sun 20 Jul 2008 10:06:57 AM UTCmsteinboStatusNone=>Workaround available
      Assigned toNone=>msteinbo

    Back to the top


    Powered by Savane 3.1-cleanup1