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

Use LC_ALL=POSIX for character ranges #15791

Closed
vbraun opened this issue Feb 6, 2014 · 9 comments
Closed

Use LC_ALL=POSIX for character ranges #15791

vbraun opened this issue Feb 6, 2014 · 9 comments

Comments

@vbraun
Copy link
Member

vbraun commented Feb 6, 2014

Component: build

Author: Volker Braun

Branch/Commit: u/jdemeyer/ticket/15791 @ be4c4ad

Reviewer: Jeroen Demeyer

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

@vbraun vbraun added this to the sage-6.2 milestone Feb 6, 2014
@vbraun
Copy link
Member Author

vbraun commented Feb 6, 2014

@vbraun
Copy link
Member Author

vbraun commented Feb 6, 2014

Last 10 new commits:

f8af7c0Merge branch 'develop' into ticket/15569
3980910Update sagenb to 0.10.8.1
dce6f58Update to version 0.10.8.2
36d088dTrac #15778: Fix broken mathjax in sagenb
add6e26Update README.txt re installation with OS X Mavericks
494c751Trac #15783: Update README.txt for OS X Mavericks
3861c7bTrac #15743: Document cancel_alarm() in alarm()
39a1978Remove simplify_radical from documentation of simplify_full
2a97c1cTrac #15693: Remove simplify_radical() from the doc of simplify_full() and full_simplify()
8be52e6Updated Sage version to 6.1.1

@vbraun
Copy link
Member Author

vbraun commented Feb 6, 2014

Author: Volker Braun

@vbraun
Copy link
Member Author

vbraun commented Feb 6, 2014

Commit: a7e6f6f

@vbraun vbraun changed the title Use LC_ALL=C for character ranges Use LC_ALL=POSIX for character ranges Feb 6, 2014
@jdemeyer
Copy link

jdemeyer commented Feb 7, 2014

@jdemeyer
Copy link

jdemeyer commented Feb 7, 2014

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link

jdemeyer commented Feb 7, 2014

comment:4

Are you sure that LC_ALL=POSIX is as portable as LC_ALL=C? In theory, they are supposed to be equal, but the latter seems to be more commonly used (therefore, probably a larger chance to be supported on various systems).

I changed it to LC_ALL=C (and rebased to 6.2.beta0). If you agree, you can set the ticket to positive_review.


New commits:

be4c4adUse posix locale for regex

@jdemeyer
Copy link

jdemeyer commented Feb 7, 2014

Changed commit from a7e6f6f to be4c4ad

@vbraun
Copy link
Member Author

vbraun commented Feb 7, 2014

comment:5

I don't know any system that understands LC_ALL=C but not POSIX. Its part of posix, after all (duh). But then I don't care, either way is fine.

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

2 participants