Closed (fixed)
Project:
Drush Make
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
19 Apr 2010 at 11:39 UTC
Updated:
6 May 2010 at 19:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
dwwSounds like a bug in drush make. Are you doing this on the command-line directly with drush, or are you using the UI on drupal.org?
Comment #2
juliangb commentedUI in drupal.org.
I first found it when trying to create a release - and then also when I tried the 'verify make file' function.
Comment #3
hunmonk commentedthis is a problem in
drush_make_validate_info_file()-- the regex doesn't support any alpha|beta|rc releases at all:Comment #4
dwwSo it's not just in the d.o-specific plugin for drush make, after all. Moving component...
Comment #5
hunmonk commentedhow's this look?
Comment #6
dwwUntested, but that visually appears fine to me.
It's case-sensitive, though, so you might want
[a-zA-Z0-9]+just in case (although our version strings are _supposed_ to be lowercase). Or, at that point, you could just use 1 or more "word" characters:\w+since it's basically equivalent except \w also matches underscores.*shrug*
Comment #7
hunmonk commented.make files are specified according to the visible version string (the kind you'd see on the title of release nodes), so for consistency i'll keep this validation the same.
Comment #8
dmitrig01 commentedfixed, thanks