Opened 9 years ago

Closed 9 years ago

#10986 closed defect (fixed)

building ecl fails in case the installed etags is actually exuberant-ctags

Reported by: fbissey Owned by: tbd
Priority: blocker Milestone: sage-4.7
Component: packages: standard Keywords:
Cc: schilly Merged in: sage-4.7.alpha5
Authors: François Bissey Reviewers: Harald Schilly, Simon King
Report Upstream: Reported upstream. Little or no feedback. Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by fbissey)

In some debian installs (and derivatives such as ubuntu) the etags program is actually to ctags from exuberant-ctags. This is incompatible with what ecl expect. the symptoms are as follow:

echo > TAGS
if test "xetags" != "x"; then \
        srcfiles=`find 
/scratch/scratch/schilly/sage/sage-4.6.2/spkg/build/ecl-11.1.1/src/src/c 
/scratch/scratch/schilly/sage/sage-4.6.2/spkg/build/ecl-11.1.1/src/src/h 
-name '*.[chd]'` && \
        etags --language=c    -o TAGS $srcfiles && \
        etags --language=none -o TAGS --append \
              --regex='/@\([-:*a-zA-z]+\)/\1/' \
              --regex='/@(defun \([-:*a-zA-z]+\)/\1/' \
              $srcfiles; \
        fi
etags: relocation error: /usr/lib/libldap_r-2.4.so.2: symbol 

or

echo > TAGS
if test "xetags" != "x"; then \
srcfiles=`find  
/storage/strogdon/gentoo/var/tmp/portage/dev-lisp/ecls-11.1.1/work/ecl-11.1.1/src/c  
/storage/strogdon/gentoo/var/tmp/portage/dev-lisp/ecls-11.1.1/work/ecl-11.1.1/src/h  
-name '*.[chd]'` && \
etags --language=c    -o TAGS $srcfiles && \
etags --language=none -o TAGS --append \
       --regex='/@\([-:*a-zA-z]+\)/\1/' \
       --regex='/@(defun \([-:*a-zA-z]+\)/\1/' \
       $srcfiles; \
fi
etags: Warning: "--language" option is obsolete; use "--language-force"  
instead
etags: "TAGS" doesn't look like a tag file; I refuse to overwrite it.

There is no way to disable the building of TAGS in the configuration and detecting that you have an etags program from emacs is complicated. See the following bug reports:

New spkg: http://spkg-upload.googlecode.com/files/ecl-11.1.1.p0.spkg

Attachments (1)

trac_10986-spkg-upgrade.patch (1.4 KB) - added by fbissey 9 years ago.
summary of change patch, for the reviewer only

Download all attachments as: .zip

Change History (8)

comment:1 Changed 9 years ago by fbissey

  • Description modified (diff)
  • Status changed from new to needs_review

Here is the new spkg. Basically I touch build/TAGS before make is called so that TAGS is never built. Bypassing any problem with a broken etags or ctags passing for etags. I will attach a patch for the reviewer with all the changes to the spkg.

Changed 9 years ago by fbissey

summary of change patch, for the reviewer only

comment:2 Changed 9 years ago by fbissey

This is now ready for review.

comment:3 Changed 9 years ago by schilly

nice, it just built successfully on the same machine where i reported the problem mentioned above!

comment:4 Changed 9 years ago by jdemeyer

  • Authors set to François Bissey
  • Reviewers set to Harald Schilly
  • Status changed from needs_review to positive_review

comment:5 Changed 9 years ago by jdemeyer

  • Reviewers changed from Harald Schilly to Harald Schilly, Simon King

comment:6 Changed 9 years ago by SimonKing

Thank you for changing it into a positive review! I was just about to do that myself.

comment:7 Changed 9 years ago by jdemeyer

  • Merged in set to sage-4.7.alpha5
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.