Merge lp://staging/~bellini666/charms/precise/wordpress/mod_rewrite into lp://staging/charms/wordpress

Proposed by Thiago Bellini
Status: Merged
Merged at revision: 73
Proposed branch: lp://staging/~bellini666/charms/precise/wordpress/mod_rewrite
Merge into: lp://staging/charms/wordpress
Diff against target: 16 lines (+6/-0)
1 file modified
hooks/config-changed (+6/-0)
To merge this branch: bzr merge lp://staging/~bellini666/charms/precise/wordpress/mod_rewrite
Reviewer Review Type Date Requested Status
Marco Ceppi (community) Approve
Review via email: mp+219440@code.staging.launchpad.net

Description of the change

It's very commom to have mod_rewrite on wordpress installations.

Also, wordpress creates .htaccess for it automatically depending on the permalink option choosen.

By enabling mod_rewrite by default on the charm, changing permalink is trivial. Otherwise, one would have to ssh to the machine just to "a2enmod rewrite".

Not to mention that, if we change the apache2 to nginx and then change back, it would need to "a2enmod rewrite" again.

To post a comment you must log in.
Revision history for this message
Marco Ceppi (marcoceppi) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: