Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Building built-in libAfterImage fails if build path contains @ #7786

Open
1 task done
eguiraud opened this issue Apr 6, 2021 · 1 comment
Open
1 task done

Building built-in libAfterImage fails if build path contains @ #7786

eguiraud opened this issue Apr 6, 2021 · 1 comment

Comments

@eguiraud
Copy link
Member

eguiraud commented Apr 6, 2021

  • Checked for duplicates

Describe the bug

The symptom is the following:

/usr/bin/mkdir -p /data/ssdext4/rdf_scaling/relwithdebinfo@master/include/libAfterImage
installing libAfterImage headers using command -
    "-m 644 <header> /data/ssdext4/rdf_scaling/relwithdebinfo@master/include/libAfterImage" :
afterimage.h afterbase.h ascmap.h asfont.h asim_afterbase.h asimage.h asimagexml.h asstorage.h asvisual.h blender.h bmp.h char2uni.h draw.h export.h imencdec.h import.h pixmap.h scanline.h transform.h ungif.h xcf.h ximage.h xpm.h xwrap.h

/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
/bin/sh: line 30: -m: command not found
gmake[3]: m: Command not found
/bin/sh: line 2: -m: command not found
gmake[3]: *** [Makefile:145: install.static] Error 127

The cause seems to be the sed substitutions at

INSTALL=`echo $INSTALL|sed -e "s@\^\.\.@${currdir}@" -e "s@^autoconf@${currdir}/autoconf@"`
(thanks @chrisburr )

@dennisklein mentioned that in homebrew they work around the issue with this patch.

@amadio suggested we should "use / as separator and escape the slashes that are part of the patterns themselves."

To Reproduce

Build ROOT with built-in libafterimage in a directory with a path that contains a @.

Additional context

Further discussion available at https://mattermost.web.cern.ch/root/pl/zszko6zhwjnsprxjk54gu5ye6h

@ChristianTackeGSI
Copy link
Collaborator

Using | is also a common solution for this kind of problem.
s|foo@bar/extra|new/value|
I yet have to see someone using a | in their paths… We'll surely get to see that once!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants