Posting this at mig5's request (somewhat belatedly).

I ran into this problem on 0.4-alpha7.

I created a new platform and got an error message along the lines of "platform failed to verify, doesn't contain a valid drupal". It turned out to be a cut and paste error, I had a space at the end of the path when entering it in the aegir ui. Obviously it's then different from the actual path and the error message is essentially correct, but mig5 suggested validating the platform form:

mig5 ‣ i think we should just check for a space, or other invalid characters, in the hook_validate of the platform form

Which would probably save some confusion and hair pulling.

Comments

adrian’s picture

I am committing a fix now which trims the space when passing it to the backend.

adrian’s picture

Category: feature » bug
Status: Active » Needs work

we should probably add validation too.

Carl Johan’s picture

so trim + validate that the filesystem path exists?

adrian’s picture

Status: Needs work » Fixed

we do validation on verify

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 26b1dd6 on dev-dns, dev-features, dev-log_directory, dev-migrate_aliases, dev-multiserver-install, dev-newsiteform, dev-nginx, dev-ssl, prod-koumbit, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-588728-views-integration, dev-1403208-new_roles, dev-helmo-3.x by adrian:
    Trim the path from extra spaces when passing to the backend - #840474
    
    

  • Commit 26b1dd6 on dev-dns, dev-features, dev-log_directory, dev-migrate_aliases, dev-multiserver-install, dev-newsiteform, dev-nginx, dev-ssl, prod-koumbit, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-588728-views-integration, dev-1403208-new_roles, dev-helmo-3.x by adrian:
    Trim the path from extra spaces when passing to the backend - #840474