[haiku-bugs] Re: [Haiku] #12710: BPackageInfo; URL Validation Checks

  • From: "apl-haiku" <trac@xxxxxxxxxxxx>
  • Date: Tue, 31 May 2016 19:32:45 -0000

#12710: BPackageInfo; URL Validation Checks
--------------------------------+----------------------------
   Reporter:  apl-haiku         |      Owner:  bonefish
       Type:  enhancement       |     Status:  new
   Priority:  normal            |  Milestone:  Unscheduled
  Component:  Kits/Package Kit  |    Version:  R1/Development
 Resolution:                    |   Keywords:
 Blocked By:                    |   Blocking:
Has a Patch:  1                 |   Platform:  All
--------------------------------+----------------------------

Comment (by apl-haiku):

 I have modified the `libgnuregex` target to produce a shared library build
 product.  This seems to then work fine.  I added a brief `ReadMe.md` file
 to help anybody else who is caught out with this problem.

 I then noticed there were some issues with the validation of the BUrl.
 For example, it seemed to be tolerant of trailing whitespace -- something
 I was trying to avoid.  It also included the port in the host field in
 some cases which is also not quite right.

 I've therefore modified the parsing logic in `BUrl` a bit to tighten it
 up.  At the same time, I have added some automated testing to cover
 scenarios around those changes so that it is safer to make further changes
 in this class.

 I was able to reproduce the problem Axel mentioned on Ubuntu; that should
 be sorted out now and has been tested on GCC 5.3.1.

 On the MacOS-X platform, the operating system makes `.DS_Store` files all
 over the place so I added a `.gitignore` entry for those at the top level.

 The commit message has been adjusted as suggested by Axel.

 I hope it is good to go!

--
Ticket URL: <https://dev.haiku-os.org/ticket/12710#comment:32>
Haiku <https://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: