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

Sage's Singular scripts should use "$@" #11774

Closed
nexttime mannequin opened this issue Sep 2, 2011 · 2 comments
Closed

Sage's Singular scripts should use "$@" #11774

nexttime mannequin opened this issue Sep 2, 2011 · 2 comments

Comments

@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 2, 2011

The Singular start-up scripts ($SAGE_ROOT/local/bin/[Ss]ingular) created by Sage currently look like this:

#!/bin/sh
Singular-3-1-1 $*

The $* has to be changed to "$@" (including the quotes), to allow spaces within parameters.

They are created by Singular's spkg-install (in create_singular_script()), so the spkg has to be changed.

Component: packages: standard

Keywords: sd34 spaces parameters

Reviewer: Jeroen Demeyer

Issue created by migration from https://trac.sagemath.org/ticket/11774

@nexttime nexttime mannequin added this to the sage-5.11 milestone Sep 2, 2011
@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@fchapoton
Copy link
Contributor

comment:5

This does not seem to me to make sense any longer. Maybe this can be closed ?

@fchapoton fchapoton removed this from the sage-6.4 milestone Oct 21, 2014
@jdemeyer
Copy link

Reviewer: Jeroen Demeyer

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

3 participants