Opened 5 years ago

Closed 5 years ago

#15678 closed defect (fixed)

R fails to build on Cygwin64 passing --large-address-aware to ld.

Reported by: jpflori Owned by:
Priority: major Milestone: sage-6.2
Component: porting: Cygwin Keywords: r cygwin
Cc: kcrisman, dimpase, tscrim, vbraun Merged in:
Authors: Jean-Pierre Flori Reviewers: Volker Braun
Report Upstream: Reported upstream. Developers deny it's a bug. Work issues:
Branch: 7015c53 (Commits) Commit: 7015c53f0d853e8708ae0f4c186cddc9920a2f5b
Dependencies: Stopgaps:

Description

This is triggered on cygwin* but makes ld fail on cygwin64.

Change History (11)

comment:1 Changed 5 years ago by jpflori

  • Authors set to Jean-Pierre Flori
  • Branch set to u/jpflori/ticket/15678
  • Keywords r cygwin added
  • Report Upstream changed from N/A to Reported upstream. No feedback yet.
  • Status changed from new to needs_review

Patch to configure.ac and configure (not automagically autoreconf, but made by hand, as the former solution produces far too much).

comment:3 Changed 5 years ago by git

  • Commit set to 7015c53f0d853e8708ae0f4c186cddc9920a2f5b

Branch pushed to git repo; I updated commit sha1. New commits:

7015c53Don't pass large-address-aware flag to ld on Cygwin64.

comment:4 follow-up: Changed 5 years ago by jpflori

  • Report Upstream changed from Reported upstream. No feedback yet. to Reported upstream. Developers deny it's a bug.

Doing my homework.

comment:5 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:6 in reply to: ↑ 4 Changed 5 years ago by kcrisman

Doing my homework.

If the word "cygwin" occurs anywhere in their source code (and the configure has that whole section), they have no right to treat you like this. Vewy wude. And why wouldn't they want to support Cygwin?

Also, I don't understand why they don't have a web-browsable source. Or actually, they do, but it's not apparently linked from the page...

comment:7 Changed 5 years ago by jpflori

Sure, that was rude. I don't plan on fighting for that.

comment:8 Changed 5 years ago by jpflori

  • Cc vbraun added

Volker: in case you're in a reviewing mood, this one is awfully simple...

comment:9 Changed 5 years ago by vbraun

  • Reviewers set to Volker Braun

comment:10 Changed 5 years ago by vbraun

you found the Grinch on the R bugzilla ;-)

comment:11 Changed 5 years ago by vbraun

  • Branch changed from u/jpflori/ticket/15678 to 7015c53f0d853e8708ae0f4c186cddc9920a2f5b
  • Resolution set to fixed
  • Status changed from needs_review to closed
Note: See TracTickets for help on using tickets.