Skip to content

Remove dead comments#414

Merged
lehins merged 5 commits into
haskell:masterfrom
Shimuuar:cleanup
Sep 9, 2021
Merged

Remove dead comments#414
lehins merged 5 commits into
haskell:masterfrom
Shimuuar:cleanup

Conversation

@Shimuuar
Copy link
Copy Markdown
Contributor

@Shimuuar Shimuuar commented Sep 8, 2021

  1. Type class is defined in D.V.Generic.Base. No reason to keep its commented out code
  2. Remove obsolete comment. When offending function (Data.Vector.Unboxed.unstream . Bundle.fromList) is compiled it produces rather straightforward core. It seems GHC fixed problem whatever it was.
  3. Typo in error message

No reason to keep its commented out code
When offending function (Data.Vector.Unboxed.unstream . Bundle.fromList) is
compiled it produces rather straightforward core.

It seems GHC fixed problem whatever it was.
@Shimuuar Shimuuar requested review from Bodigrim and lehins September 8, 2021 15:25
Copy link
Copy Markdown
Contributor

@lehins lehins left a comment

Choose a reason for hiding this comment

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

I wanted to remove that stuff for the longest time, but kept forgetting about. Thank you for finally getting rid of it.

Some weird issue on CI, might be borked cache. I restarted it, maybe it will fix itself

@Shimuuar
Copy link
Copy Markdown
Contributor Author

Shimuuar commented Sep 8, 2021

Some weird issue on CI, might be borked cache. I restarted it, maybe it will fix itself

It didn't. GHC 8.6.5 fails quite reliably on ubuntu with

vector-doctest: /usr/local/.ghcup/ghc/8.6.5/bin/ghc-8.6.5: getPermissions:getFileStatus: does not exist (No such file or directory)

@lehins
Copy link
Copy Markdown
Contributor

lehins commented Sep 8, 2021

This is weird: .ghcup/ghc/8.6.5/bin/ghc-8.6.5

ghcup? I thought we are using PPA for installing ghc

@Shimuuar
Copy link
Copy Markdown
Contributor Author

Shimuuar commented Sep 9, 2021

ghcup? I thought we are using PPA for installing ghc

I'm not sure it's ghcup. But I guess it is. PPA does not for work for windows/mac and we don't treat them specially. I'll try to work around cache(?) problem

CI jobs fail with weird error:

> vector-doctest: /usr/local/.ghcup/ghc/8.6.5/bin/ghc-8.6.5:
> getPermissions:getFileStatus: does not exist (No such file or directory)

Let try to change cache key in order to avoid using possibly broken cache
@lehins
Copy link
Copy Markdown
Contributor

lehins commented Sep 9, 2021

I get a feeling this recent change might have something to do with it:

https://github.com/actions/virtual-environments/pull/4016/files

@Shimuuar
Copy link
Copy Markdown
Contributor Author

Shimuuar commented Sep 9, 2021

Maybe. Looks like dropping cache did the trick

@lehins lehins merged commit 2dd6741 into haskell:master Sep 9, 2021
@Shimuuar Shimuuar deleted the cleanup branch September 11, 2021 10:04
lehins pushed a commit that referenced this pull request Sep 21, 2021
Remove dead comments and add versioning key to caching on CI:

* Type class is defined in D.V.Generic.Base

No reason to keep its commented out code

* Fix error message

* Remove obsolete comment

When offending function (Data.Vector.Unboxed.unstream . Bundle.fromList) is
compiled it produces rather straightforward core.

It seems GHC fixed problem whatever it was.

* There's another dead comment

* Change key for GA cache

CI jobs fail with weird error:

> vector-doctest: /usr/local/.ghcup/ghc/8.6.5/bin/ghc-8.6.5:
> getPermissions:getFileStatus: does not exist (No such file or directory)

Let try to change cache key in order to avoid using possibly broken cache
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