[haiku-bugs] Re: [Haiku] #18115: "Checksum error" while attempting to update packages from the Haiku repo.

  • From: "Haiku" <trac@xxxxxxxxxxxx>
  • To: undisclosed-recipients: ;
  • Date: Sun, 04 Dec 2022 10:28:49 -0000

#18115: "Checksum error" while attempting to update packages from the Haiku 
repo.
------------------------+----------------------
  Reporter:  bipolar    |      Owner:  nobody
      Type:  bug        |     Status:  new
  Priority:  high       |  Milestone:  R1/beta4
 Component:  - General  |    Version:  R1/beta4
Resolution:             |   Keywords:
Blocked By:             |   Blocking:
  Platform:  All        |
------------------------+----------------------
Comment (by bipolar):

 Slight side-track here but... notice how pkgman output/error messages are
 a bit confusing here.

 I've notice two things:

 1- There are some missing `\n` between:
  a- the last `Checksum error:` line.

  b- the `Refreshing repository "Haiku" failed` message.

  c- the `Validating checksum for Haiku...: Bad data` message.

 A quick look at that code makes it seem (to me) that the messages expect
 the last '\n' to be added by the code actually doing the output (as is
 done by ``UserInteractionHandler::Warn()` implementation on pkgman's
 'PackageManager::Warn()').

 But either I get lost too easily (most likely), or I couldn't find
 where/why it fails to add those '\n' as it should.

 2- There's seems to be a `Validating checksum for <reponame>` message for
 two different stages of validation, the one for the URL checksum
 (performed by `BRefreshRepositoryRequest::CreateInitialJobs()`) and for
 the "repo cache" checksum (done by
 `BRefreshRepositoryRequest::_FetchRepositoryCache()`.

 I guess the second should be changed, so in this case, instead of getting:

 `Validating checksum for Haiku...: Bad data`,

 we should have something like:

 `Validating checksum for Haiku repo cache...: Bad data`

 no?
-- 
Ticket URL: <https://dev.haiku-os.org/ticket/18115#comment:8>
Haiku <https://dev.haiku-os.org>
The Haiku operating system.

Other related posts: