Opened 7 months ago
Closed 6 months ago
#30608 closed defect (duplicate)
fix symmetrica for xcode 12
Reported by:  dimpase  Owned by:  

Priority:  blocker  Milestone:  sageduplicate/invalid/wontfix 
Component:  packages: standard  Keywords:  
Cc:  mkoeppe, jhpalmieri, mjo  Merged in:  
Authors:  Dima Pasechnik  Reviewers:  John Palmieri 
Report Upstream:  N/A  Work issues:  
Branch:  u/dimpase/packages/symmetrica/xcode12fix (Commits, GitHub, GitLab)  Commit:  dfa4ab4ef70549263ff3b243ea30e38425ac6231 
Dependencies:  Stopgaps: 
Description
Change History (17)
comment:1 Changed 7 months ago by
 Branch set to u/dimpase/packages/symmetrica/xcode12fix
 Cc mkoeppe jhpalmieri added
 Commit set to dfa4ab4ef70549263ff3b243ea30e38425ac6231
 Status changed from new to needs_review
comment:2 Changed 7 months ago by
this builds.
comment:3 Changed 7 months ago by
spkg.rst should be updated  some links are broken.
comment:4 Changed 7 months ago by
I just saw one broken link. Should I push this change?

build/pkgs/symmetrica/SPKG.rst
diff git a/build/pkgs/symmetrica/SPKG.rst b/build/pkgs/symmetrica/SPKG.rst index bd82eda45f..3d0213ddba 100644
a b University of Bayreuth. It has routines to handle the following topics 18 18  operations of finite groups. 19 19  ordinary representation theory of Hecke algebras of type A_n 20 20 21 For more details check http://www.symmetrica.de (currently redirects to22 http://www.algorithm.unibayreuth.de/en/research/SYMMETRICA )21 For more details check 22 http://www.algorithm.unibayreuth.de/en/research/SYMMETRICA. 23 23 24 24 License 25 25 
comment:6 followup: ↓ 7 Changed 7 months ago by
Just upgrade it, our forked upstream version is far ahead. I'd like to make a 3.0.2 release sometime soon with the pkgconfig file, too. Additional brain power (and maybe German language skills) would be helpful on https://gitlab.com/sagemath/symmetrica//merge_requests/3 which is the only remaining item that I have no idea what to do with.
comment:7 in reply to: ↑ 6 ; followup: ↓ 8 Changed 7 months ago by
comment:8 in reply to: ↑ 7 Changed 7 months ago by
Replying to jhpalmieri:
Replying to mjo:
Just upgrade it, our forked upstream version is far ahead.
Please prepare a branch at #29061 for us to test.
I've posted a branch on #29061 it looks more attractive than this one, for sure.
comment:9 Changed 7 months ago by
We should use either this one or #29061, but not both. (For example, I think that #29061 merges cleanly on top of this one, but then the patch here does not apply to the 3.0.1 tarball.) Both are marked as blockers. Which route should we take? My preference would be: if we can test #29061 on a lot of platforms quickly, we do that. If we can't, then let's use this one now and #29061 in the next release.
comment:10 Changed 7 months ago by
 Milestone changed from sage9.2 to sageduplicate/invalid/wontfix
let's go with #29061 then, OK?
comment:11 Changed 7 months ago by
Okay with me.
comment:12 Changed 7 months ago by
But I'm not going to mark this as positive review until that one is ready. No need to close this one yet.
comment:13 Changed 7 months ago by
 Reviewers set to John Palmieri
comment:14 Changed 7 months ago by
comment:15 Changed 7 months ago by
as #29061 is ready, this one can get a positive review IMHO.
comment:16 Changed 7 months ago by
 Status changed from needs_review to positive_review
Okay, we can always return to it if we absolutely need to.
comment:17 Changed 6 months ago by
 Resolution set to duplicate
 Status changed from positive_review to closed
New commits:
xcode12 fixes for symmetrica