Skip to content

Adapt to changes in Distribution.Version. #3967

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

Merged
merged 1 commit into from
Oct 9, 2016

Conversation

kosmikus
Copy link
Contributor

@kosmikus kosmikus commented Oct 9, 2016

The DEBUG_TRACETREE debugging feature of cabal-install
still had a call to 'showVersion' in it, which has been
removed by recent changes to 'Distribution.Version'.

The DEBUG_TRACETREE debugging feature of cabal-install
still had a call to 'showVersion' in it, which has been
removed by recent changes to 'Distribution.Version'.
@mention-bot
Copy link

@kosmikus, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edsko, @grayjay and @BardurArantsson to be potential reviewers.

@phadej
Copy link
Collaborator

phadej commented Oct 9, 2016

partially related, Show instance for Version could print something more helpful than PV0 122452452.

@phadej
Copy link
Collaborator

phadej commented Oct 9, 2016

ping @hvr

Copy link
Collaborator

@phadej phadej left a comment

Choose a reason for hiding this comment

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

LGTM

@23Skidoo
Copy link
Member

23Skidoo commented Oct 9, 2016

I canceled the CI builds, since our CI currently doesn't exercise this code path (which is why the issue went unnoticed).

@23Skidoo 23Skidoo merged commit af422f7 into haskell:master Oct 9, 2016
@23Skidoo
Copy link
Member

23Skidoo commented Oct 9, 2016

Merged, thanks!

@dcoutts
Copy link
Contributor

dcoutts commented Oct 11, 2016

@hvr I agree with @phadej, it'd be more helpful to maintain some abstraction in the Show instance, e.g. have show produce "mkVersion [1,2,0,0]"

@hvr
Copy link
Member

hvr commented Oct 11, 2016

@dcoutts the auto-derived Show instance was actually an oversight... but it appears that's been taken care of by #3973 :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants