It worked before alpha5, but now seems to be missing.

Attached below patch for provision HEAD.

CommentFileSizeAuthor
#1 provision_tmp_b.patch1.89 KBomega8cc
provision_tmp.patch1.42 KBomega8cc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

omega8cc’s picture

omega8cc CreditAttribution: omega8cc commented
Title: Missing /files/tmp in d6 required for enabled css/js aggregation » Missing /files/tmp in d6
Category: feature » bug
FileSize
provision_tmp_b.patch1.89 KB

It seems there is more than just /files/tmp missing in HEAD.
Not sure about d7 changes between alpha5 and HEAD, but for d6
here is more complete patch attached.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

It seems the d6 stuff was taken out in this commit.

Best check with anarcat, as this appears deliberate. I was about to say, that since the files dir is created and is writable by the webserver, Drupal 6 automatically creates these directories on demand. As far as I know anyway. So I'm guessing that was the reasoning here (see the commit message), and it makes sense to me.

This is different to Drupal7 in that stuff actually starts to malfunction (such as user picture uploads) if certain subdirectories aren't created upon install.

omega8cc’s picture

omega8cc CreditAttribution: omega8cc commented

In d6 /files/tmp is not created automatically, at least not by core css/js aggregator and has to be created by provision module. Not sure about pictures/images there, but the /files/tmp is required for successful install of vanilla d6.

anarcat’s picture

anarcat CreditAttribution: anarcat commented
Status: Active » Fixed

yeah, sorry, my bad, committed a fix.

Status: Fixed » Closed (fixed)

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

  • Commit 2cae278 on debian, dev-dns, dev-envobject, dev-koumbit, dev-log_directory, dev-migrate_aliases, dev-multiserver-install, dev-newhooks, dev-nginx, dev-ports, dev-purgebackup, dev-restore, dev-services, dev-simplerinstaller, dev-site_rename, dev-ssl, dev_716166_apache_conf, dev_dns, dev_server_verify, prod-koumbit, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x by anarcat: 
    #719704 - create properly the tmp directory again