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

Don't make package info records immutable #2597

Merged
merged 1 commit into from
Jul 3, 2018

Conversation

fingolfin
Copy link
Member

This fixes loading of float and JuliaInterface in master, as described
in issue #2568, at least im regular GAP. For HPC-GAP, things remain
unchanged for now.

This does not yet fix that issue, as HPC-GAP remains affected. See that issue for ideas on how to properly solve the underlying concern.

This fixes loading of float and JuliaInterface in master, as described
in issue gap-system#2568, at least im regular GAP. For HPC-GAP, things remain
unchanged for now.
@fingolfin fingolfin added regression A bug that only occurs in the branch, not in a release topic: packages issues or PRs related to package handling, or specific to a package (for packages w/o issue tracker) topic: library release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes labels Jul 2, 2018
@codecov
Copy link

codecov bot commented Jul 2, 2018

Codecov Report

Merging #2597 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2597      +/-   ##
==========================================
- Coverage   75.34%   75.33%   -0.01%     
==========================================
  Files         442      442              
  Lines      228957   228958       +1     
==========================================
- Hits       172498   172496       -2     
- Misses      56459    56462       +3
Impacted Files Coverage Δ
lib/package.gi 69.8% <100%> (+0.01%) ⬆️
src/iostream.c 62.35% <0%> (-1.15%) ⬇️

@fingolfin fingolfin added this to the GAP 4.10.0 milestone Jul 2, 2018
Copy link
Member

@olexandr-konovalov olexandr-konovalov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK with this as an intermediate measure.

@fingolfin fingolfin merged commit 35f246a into gap-system:master Jul 3, 2018
@fingolfin fingolfin deleted the mh/fix-float branch July 3, 2018 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression A bug that only occurs in the branch, not in a release release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes topic: library topic: packages issues or PRs related to package handling, or specific to a package (for packages w/o issue tracker)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants