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

Backport filter --allow-depending-on-private-libs #6838

Merged

Conversation

fgaz
Copy link
Member

@fgaz fgaz commented May 23, 2020

In #6836 I noticed that I forgot to filter out the --allow-depending-on-private-libs flag for old Cabals. With that patch it becomes necessary to have the filtering, but I guess having it in 3.2 would be good too?

@fgaz fgaz added this to the 3.2.1.0 milestone May 23, 2020
@fgaz fgaz merged commit fae6c79 into haskell:3.2 May 23, 2020
@fgaz fgaz deleted the backport-filter-allow-depending-on-private-libs branch May 23, 2020 11:25
@phadej
Copy link
Collaborator

phadej commented May 23, 2020

Sorry, I didn't notice this patch was to 3.2 branch and not master. Is this change in master. Which commit introduced it?

@fgaz
Copy link
Member Author

fgaz commented May 23, 2020

It isn't in master yet. It's part of #6836. By the way, could you restart https://ci.appveyor.com/project/23Skidoo/cabal/builds/33067141 so we can merge that too? I'm pretty sure that failure is just random

@phadej
Copy link
Collaborator

phadej commented May 23, 2020

Let's avoid backporting stuff which is not merged yet. Can #6836 merged as CI is done?

Also you merged this even GitHub actions Windows 8.6.5 job was broken. Which is something I'm actively working on in #6828. I work hard to have especially release branches green, please don't merge if CI is not green.

I should enforced the requirement for CI to be green for 3.2 branch too. Now it is.

@fgaz
Copy link
Member Author

fgaz commented May 23, 2020

Yes, when #6836 is green we can add the changelog entry and merge.

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.

2 participants