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

findProgramOnSearchPath for Cabal 1.24.0, findProgramLocation for anything below #22

Merged
merged 1 commit into from Oct 22, 2016

Conversation

mchaver
Copy link
Contributor

@mchaver mchaver commented Jun 16, 2016

No description provided.

@Rockfordal
Copy link

tnx alot! had to learn how to point my stack.yaml to mchaver's repo to be able to compile a simple servant project. im a beginner, but why hasnt the needed fixes been pulled in or masked?

@niteria
Copy link

niteria commented Sep 12, 2016

Ping. It's almost 3 months since the fix was proposed.

{- OPTIONS_GHC -Wall #-}

#ifndef MIN_Version_Cabal
#define MIN_Version_Cabal(x,y,z) 0
Copy link

Choose a reason for hiding this comment

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

Capitalization doesn't match up. Is that intentional?

@DylanMeeus
Copy link

Any luck on getting this merged in?

@3noch
Copy link

3noch commented Sep 26, 2016

At this point I highly recommend that someone who deals with MySQL and Haskell on a regular basis ask @bos if he can become a contributor. If that goes nowhere I'd suggest forking the repo, applying all these improvements, pushing it to Hackage (mysql-haskell or something?) and notifying the community that this package is unofficially deprecated.

@jtdaugherty
Copy link

@bos - this looks uncontroversial to me, and it would help a lot of folks. Can we get this merged, please?

@paul-rouse
Copy link
Owner

Thanks - I am now maintaining this package. I appears the mismatched capitalisation is wrong, but I'll merge and then fix it.

@paul-rouse paul-rouse merged commit 6600179 into paul-rouse:master Oct 22, 2016
@paul-rouse
Copy link
Owner

Now released on hackage as 0.1.2

@jtdaugherty
Copy link

@paul-rouse Thank you!

@3noch
Copy link

3noch commented Oct 22, 2016

@paul-rouse Thank you for taking over this package and making progress! Thank you @bos for all the work you've put in over the years!

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

Successfully merging this pull request may close these issues.

None yet

7 participants