Skip to content

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

Merged
fgaz merged 1 commit intohaskell:3.2from
fgaz:backport-filter-allow-depending-on-private-libs
May 23, 2020
Merged

Backport filter --allow-depending-on-private-libs#6838
fgaz merged 1 commit intohaskell:3.2from
fgaz:backport-filter-allow-depending-on-private-libs

Conversation

@fgaz
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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