Ticket #1383 (closed feature: fixed)

Opened 5 years ago

Last modified 5 years ago

[PATCH] add a cli option to set Parrot's hash seed

Reported by: cotto Owned by: cotto
Priority: minor Milestone:
Component: none Version: 1.9.0
Severity: low Keywords:
Cc: Language:
Patch status: Platform:

Description (last modified by coke) (diff)

The attached patch adds --hash-seed to parrot's cli options. When set to a valid hex value, interp->hash_seed will contain that value instead of a sorta-random value based on the current time. The value of this is to speed debugging of failures that appear to be related to hash ordering. Coke found such a bug in partcl recently.

The patch is pretty simple, but I thought I'd put this out there before committing in the likely case that I've missed something.

Attachments

hash_seed_option.patch Download (1.9 KB) - added by cotto 5 years ago.
first version of the patch

Change History

Changed 5 years ago by cotto

first version of the patch

Changed 5 years ago by coke

  • description modified (diff)

Changed 5 years ago by cotto

  • priority changed from normal to minor

Using this patch causes strange and wonderful failures when running nqp-rx scripts and anything non-trivial. This patch shouldn't be applied until I figure out what kinds of shenanigans are causing these problems.

Changed 5 years ago by cotto

  • status changed from new to closed
  • resolution set to fixed

After bacek moved the parseopts code out of imcc, the mysterious failures that this patch previously introduced disappeared. I've applied it as r43935 and am marking this ticket as fixed.

Note: See TracTickets for help on using tickets.