! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
configure should check for gnu sed
Status:
RESOLVED: FIXED

Comments

Description Landry Breuil editbugs 2012-04-13 16:04:48 CEST
sed -i is a gnu extension and is not present in OpenBSD's sed.

thunar hardcodes its use in thunar/Makefile.am:
thunar/Makefile.am:              && sed -i -e 's/org_freedesktop_thumbnails_Thumbnailer1/thunar_thumbnailer_proxy/g' \
thunar/Makefile.am:              && sed -i -e 's/org_freedesktop_thumbnails_Cache1/thunar_thumbnail_cache_proxy/g' \

configure should check for SED program (so that we can override it with SED=/usr/local/bin/gsed on OpenBSD) and use $(SED) in Makefile.am instead.

A way to check for it is :

dnl AC_PROG_SED is only avaliable in recent autoconf versions.
dnl Use AC_CHECK_PROG instead if AC_PROG_SED is not present.
ifdef([AC_PROG_SED],
      [AC_PROG_SED],
      [AC_CHECK_PROG(SED, sed, sed)])

(taken from http://svn.apache.org/repos/asf/apr/apr/trunk/configure.in)

AC_PROG_SED macro is available since autoconf 2.62.
Comment 1 Nick Schermer editbugs 2012-04-13 18:05:30 CEST
Fixed in 7c194fa.

Bug #8686

Reported by:
Landry Breuil
Reported on: 2012-04-13
Last modified on: 2012-04-13

People

Assignee:
Jannis Pohlmann
CC List:
1 user

Version

Version:
unspecified

Attachments

Additional information