profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/Ericson2314/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
John Ericson Ericson2314 @ObsidianSystems New York *x* should be designed not by piling feature on top of feature, but by removing the weaknesses and restrictions that make additional features appear necessary.

alpmestan/ghc.nix 101

Nix (shell) expression for working on GHC

ekmett/bytes 20

Serialization primitives that work with both cereal and binary.

Ericson2314/clash-prelude-quickcheck 6

QuickCheck instances for various types in the CλaSH Prelude

Ericson2314/disass 2

A conservative, sound disassembler designed to facilitate easy reassembly

Ericson2314/enet 2

ENet reliable UDP networking library

Ericson2314/APB-signiture-stats 1

for Command & Conquer: Red Alert: A Path Beyond. Design by APB_ICE

Ericson2314/agda-scratchpad 0

Messing with Agda

Ericson2314/alloc-wg 0

Attempt of collection several proposals of the allocators-wg

issue closedNixOS/nixpkgs

Move backwards-compatible yet cross-unsafe mkDerivation behavior behind a single switch

During a native build, stdenv has some behaviors that are inappropriate from a cross-compilation perspective yet were kept to reduce churn and increase compatibility (e.g. packages in buildInputs being in PATH during the build). We should have these "cheats" behind a single flag so we can easily track (ideally in CI) how much we need to fix before we can disable it altogether.

Action item from #35543

closed time in an hour

shlevy
PullRequestReviewEvent

Pull request review commentsimonmar/happy

Create Tabular package

 Generating info files. (c) 1993-2001 Andy Gill, Simon Marlow ----------------------------------------------------------------------------- -> module Info (genInfoFile) where+> module Happy.Tabular.Info (genInfoFile) where -> import Paths_happy            ( version )-> import LALR                   ( Lr0Item(..), Goto(..), LRAction(..), ActionTable, GotoTable )-> import GenUtils               ( str, interleave, interleave' ) > import Data.Set ( Set ) > import qualified Data.Set as Set hiding ( Set ) > import Happy.Grammar+> import Happy.Tabular.LALR   ( Lr0Item(..), LRAction(..), Goto(..), GotoTable, ActionTable )+> import Paths_happy_tabular  ( version )

Return the String -> String in this module and put it altogether downstream.

knothed

comment created time in 9 hours

push eventreflex-frp/reflex-platform

John Ericson

commit sha 704a3afd5d93c1582274f155f7e5b434d8813df3

Document last PR in the change log

view details

John Ericson

commit sha 7bbef459ad1446979a126ad0c8592f97f36399a0

Merge remote-tracking branch 'origin/develop' into ghc-8.10-ios

view details

push time in 12 hours

push eventreflex-frp/reflex-platform

John Ericson

commit sha 704a3afd5d93c1582274f155f7e5b434d8813df3

Document last PR in the change log

view details

push time in 12 hours

PR opened reflex-frp/reflex-platform

GHC 8.10 iOS supporrt
+114 -17

0 comment

9 changed files

pr created time in 12 hours

Pull request review commentsimonmar/happy

Create Tabular package

 In hindsight, this was probably a bad idea. > firstStartTok   = 3 > dummyTok        = 2 > errorTok        = 1-> epsilonTok      = 0\ No newline at end of file+> epsilonTok      = 0++-----------------------------------------------------------------------------+Replace $$ with an arbitrary string, being careful to avoid ".." and '.'.++> mapDollarDollar :: String -> Maybe (String -> String)+> mapDollarDollar code0 = go code0 ""

OK that makes sense.

knothed

comment created time in 13 hours

PullRequestReviewEvent

Pull request review commentsimonmar/happy

Create Tabular package

 Generating info files. (c) 1993-2001 Andy Gill, Simon Marlow ----------------------------------------------------------------------------- -> module Info (genInfoFile) where+> module Happy.Tabular.Info (genInfoFile) where -> import Paths_happy            ( version )-> import LALR                   ( Lr0Item(..), Goto(..), LRAction(..), ActionTable, GotoTable )-> import GenUtils               ( str, interleave, interleave' ) > import Data.Set ( Set ) > import qualified Data.Set as Set hiding ( Set ) > import Happy.Grammar+> import Happy.Tabular.LALR   ( Lr0Item(..), LRAction(..), Goto(..), GotoTable, ActionTable )+> import Paths_happy_tabular  ( version )

I'm interested in @int-index's two cents. If we use the exe version I think we should also move header part like I was thinking.

knothed

comment created time in 13 hours

PullRequestReviewEvent

pull request commentsimonmar/happy

Create Tabular package

@knothed That's better! Then I'd say leave where it was in this commit, and then convert to code comments in subsequent commit.

knothed

comment created time in 13 hours

issue commentNixOS/nix

Create a bottom up tutorial on Nix

Have you seen https://nixos.org/guides/nix-pills/ ?

Guisanpea

comment created time in a day

issue commenthaskell/cabal

Plans that use bootstrapping to break cycles

@Mikolaj Yes, that required for this to work, thanks for pointing out.

Ericson2314

comment created time in a day

pull request commentghc-proposals/ghc-proposals

Codify proposals in an "Apsirational User's Guide"

@goldfirere

And, in this case, I think we have bigger and better problems to solve.

Not to put you on the spot, but I am curious what you are thinking we should be doing. This stuff seems quite worthwhile to me, and even if the cost is 1 person-year as you fear, time well spent. I generally think we need a lot more consolidation / cleanup / taking stock of what we have / improving processes -type work, and this is just one example of that. I certainly more urgency in that over the isolated new features that we generally see being proposed.

Generally with ghc-proposals the authors bring their own budget, and so we don't need to worry about the relative prioritization of goals. But for something like this that I certainly cannot do alone, and which you evidently like in the abstract but don't think it is worth the costs, I think it makes sense to discuss priorities more broadly.

Ericson2314

comment created time in a day

issue openedNixOS/nix

Nix might not be trying to substitute enough with derivations it cannot build.

Describe the bug

I do nix-store -r /nix/store/asdfasdfasfd-a-darwin.drv, which is input-addressed, and which produces a path I can substitute. But instead of substituting the path, Nix complains it cannot produce the derivation.

Steps To Reproduce

I noticed this doing nix-instantiate --argstr system x86_64-darwin -A iosReflexTodomvc-8_6 with github.com/reflex-frp/reflex-platform, and hitting things that were IFD'd, but I do not have a more minimal repro yet.

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior

Nix substitutes the path, and doesn't even try to build the derivation.

nix-env --version output

nix (Nix) 2.3.15, but @fendor hit this with a Nix 2.4pre too.

Additional context

Don't think so.

created time in a day

pull request commentNixOS/nix

New "indexed" installable syntax: `<drvPath>!<outputName>`

I should also mention that nix-store -r /nix/store/3v8v9alqxfhif646mrvrafv2hmw60ypx-cabal2nix-reflex.drv!out is already valid syntax, hah.

Ericson2314

comment created time in a day

push eventreflex-frp/reflex-platform

John Ericson

commit sha 154bcf0ff08a8bcc7ed0f56bb09a79870985499f

Merge pull request #750 from reflex-frp/add-ghc-8.10-proper-only Try just addding ghc 8.10 native, no mobile or ghcjs

view details

John Ericson

commit sha 5ca2375cbe16f8e5b7c5955a297c5749f3f469e8

Merge remote-tracking branch 'origin/develop' into ghc-8.10-ios

view details

John Ericson

commit sha 8701a1967fe3628827de1e1d5b0441fa2bc3d40f

Merge branch 'ghc-8.10-ios' into ghc-8.10

view details

push time in a day

push eventreflex-frp/reflex-platform

John Ericson

commit sha 154bcf0ff08a8bcc7ed0f56bb09a79870985499f

Merge pull request #750 from reflex-frp/add-ghc-8.10-proper-only Try just addding ghc 8.10 native, no mobile or ghcjs

view details

John Ericson

commit sha 5ca2375cbe16f8e5b7c5955a297c5749f3f469e8

Merge remote-tracking branch 'origin/develop' into ghc-8.10-ios

view details

push time in a day

push eventreflex-frp/reflex-platform

John Ericson

commit sha efc5a6b34db80929c9111b0063c314d8c41106d2

Try adding GHC 8.10 iOS support

view details

John Ericson

commit sha 7c96edfb48939b295d2207b28bf0449aff14efb2

Merge branch 'ghc-8.10-ios' into ghc-8.10

view details

push time in a day

create barnchreflex-frp/reflex-platform

branch : ghc-8.10-ios

created branch time in a day

delete branch reflex-frp/reflex-platform

delete branch : add-ghc-8.10-proper-only

delete time in a day

push eventreflex-frp/reflex-platform

John Ericson

commit sha c95c095a8933eb9065a9af971ea7fd8517f43dfa

Try just addding ghc 8.10 native, no mobile or ghcjs

view details

John Ericson

commit sha 8a409205c622c44ae3d5cda9f6e59400af3cfdfd

Oops, do need cabal-macosx bump

view details

John Ericson

commit sha 154bcf0ff08a8bcc7ed0f56bb09a79870985499f

Merge pull request #750 from reflex-frp/add-ghc-8.10-proper-only Try just addding ghc 8.10 native, no mobile or ghcjs

view details

push time in a day

pull request commentreflex-frp/reflex-platform

Try just addding ghc 8.10 native, no mobile or ghcjs

Woo it passed!

Ericson2314

comment created time in a day

issue commenthaskell/cabal

Plans that use bootstrapping to break cycles

I tried with https://github.com/simonmar/happy/pull/212/files using a different name to avoid this and https://github.com/haskell/cabal/issues/3502, but then I hit there there is no way to say "use happy-boot for .ly files". It's rather alpha-equivalence-violating, but the dep will only work if it's named happy.

Ericson2314

comment created time in a day

issue commenthaskell/cabal

Plans that use bootstrapping to break cycles

@gbaz same issues happens with build-tool-depends. Glad build-tools is going away, though!

Ericson2314

comment created time in a day

push eventreflex-frp/reflex-platform

John Ericson

commit sha 8a409205c622c44ae3d5cda9f6e59400af3cfdfd

Oops, do need cabal-macosx bump

view details

John Ericson

commit sha 1f1950084ffe62a101f54e5849472edd60123864

Merge branch 'add-ghc-8.10-proper-only' into ghc-8.10

view details

push time in a day

push eventreflex-frp/reflex-platform

John Ericson

commit sha 8a409205c622c44ae3d5cda9f6e59400af3cfdfd

Oops, do need cabal-macosx bump

view details

push time in a day

Pull request review commentreflex-frp/reflex-platform

Try just addding ghc 8.10 native, no mobile or ghcjs

 {   "owner": "reflex-frp",   "repo": "reflex-ghci",-  "branch": "release/0.1.5.2",+  "branch": "ts@ghc-8.10",

Good catch. I would say we can get this PR working or even merge first, but yes we should release that before releasing reflex-platform with this PR.

Ericson2314

comment created time in 2 days