[GHC] #13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling)
#13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: (none) Type: bug | Status: new Priority: high | Milestone: 8.2.1 Component: ghc-pkg | Version: 8.2.1-rc2 Keywords: | Operating System: Unknown/Multiple Architecture: | Type of failure: None/Unknown Unknown/Multiple | Test Case: | Blocked By: Blocking: | Related Tickets: Differential Rev(s): | Wiki Page: -------------------------------------+------------------------------------- If I register two internal libraries from the same package, they clobber each other. They should not do this. This regression was probably introduced by 579bb7669f40ed01841dd197ee535cf26fa19580. Working on a fix. -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13703> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: (none) Type: bug | Status: patch Priority: high | Milestone: 8.2.1 Component: ghc-pkg | Version: 8.2.1-rc2 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple Type of failure: None/Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D3590 Wiki Page: | -------------------------------------+------------------------------------- Changes (by bgamari): * status: new => patch * differential: => Phab:D3590 -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13703#comment:1> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: (none) Type: bug | Status: patch Priority: high | Milestone: 8.2.1 Component: ghc-pkg | Version: 8.2.1-rc2 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple Type of failure: None/Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D3590 Wiki Page: | -------------------------------------+------------------------------------- Comment (by Edward Z. Yang <ezyang@…>): In [changeset:"d9e9a9b3016a05e6153de3803998877f91c6cdf4/ghc" d9e9a9b/ghc]: {{{ #!CommitTicketReference repository="ghc" revision="d9e9a9b3016a05e6153de3803998877f91c6cdf4" Fix #13703 by correctly using munged names in ghc-pkg. Summary: Cabal internal libraries are implemented using a trick, where the 'name' field in ghc-pkg registration file is munged into a new form to keep each internal library looking like a distinct package to ghc-pkg and other tools; e.g. the internal library q from package p is named z-p-z-q. Later, Cabal library got refactored so that we made a closer distinction between these "munged" package names and the true package name of a package. Unfortunately, this is an example of a refactor for clarity in the source code which ends up causing problems downstream, because the point of "munging" the package name was to make it so that ghc-pkg and similar tools transparently used MungedPackageName whereever they previously used PackageName (in preparation for them learning proper syntax for package name + component name). Failing to do this meant that internal libraries from the same package (but with different names) clobber each other. This commit search-replaces most occurrences of PackageName in ghc-pkg and turns them into MungedPackageName. Otherwise there shouldn't be any functional differenes. Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu> Test Plan: validate Reviewers: bgamari, austin Subscribers: rwbarton, thomie GHC Trac Issues: #13703 Differential Revision: https://phabricator.haskell.org/D3590 }}} -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13703#comment:2> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: (none) Type: bug | Status: merge Priority: high | Milestone: 8.2.1 Component: ghc-pkg | Version: 8.2.1-rc2 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple Type of failure: None/Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D3590 Wiki Page: | -------------------------------------+------------------------------------- Changes (by bgamari): * status: patch => merge -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13703#comment:3> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: (none) Type: bug | Status: closed Priority: high | Milestone: 8.2.1 Component: ghc-pkg | Version: 8.2.1-rc2 Resolution: fixed | Keywords: Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple Type of failure: None/Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D3590 Wiki Page: | -------------------------------------+------------------------------------- Changes (by bgamari): * status: merge => closed * resolution: => fixed Comment: Merged to `ghc-8.2` as 8054a7435dc15bc45c04adabe098a78ff6d6a7eb. -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13703#comment:4> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: (none) Type: bug | Status: new Priority: highest | Milestone: 8.2.1 Component: ghc-pkg | Version: 8.2.1-rc2 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple Type of failure: None/Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D3590 Wiki Page: | -------------------------------------+------------------------------------- Changes (by bgamari): * priority: high => highest * status: closed => new * resolution: fixed => Comment: Judging from [[https://github.com/haskell/cabal/pull/4529|this Cabal bug]] it sounds like this is still broken. ezyang, do you think you could investigate? -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13703#comment:5> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: ezyang Type: bug | Status: new Priority: highest | Milestone: 8.2.1 Component: ghc-pkg | Version: 8.2.1-rc2 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple Type of failure: None/Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D3590 Wiki Page: | -------------------------------------+------------------------------------- Changes (by bgamari): * owner: (none) => ezyang -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13703#comment:6> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: ezyang Type: bug | Status: new Priority: highest | Milestone: 8.2.1 Component: ghc-pkg | Version: 8.2.1-rc2 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple Type of failure: None/Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D3590 Wiki Page: | -------------------------------------+------------------------------------- Comment (by ezyang): Nope, the remaining test failures are actually all just wobbliness in test suite. This bug is fixed! -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13703#comment:7> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#13703: Internal libraries regression in GHC 8.2 (ghc-pkg handling) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: ezyang Type: bug | Status: closed Priority: highest | Milestone: 8.2.1 Component: ghc-pkg | Version: 8.2.1-rc2 Resolution: fixed | Keywords: Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple Type of failure: None/Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D3590 Wiki Page: | -------------------------------------+------------------------------------- Changes (by ezyang): * status: new => closed * resolution: => fixed -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13703#comment:8> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
participants (1)
-
GHC