Merge lp://qastaging/~lynxman/ubuntu/oneiric/apache2/fixeddefaultsite into lp://qastaging/ubuntu/oneiric/apache2

Proposed by Marc Cluet
Status: Work in progress
Proposed branch: lp://qastaging/~lynxman/ubuntu/oneiric/apache2/fixeddefaultsite
Merge into: lp://qastaging/ubuntu/oneiric/apache2
Diff against target: 30 lines (+12/-1)
2 files modified
debian/changelog (+11/-0)
debian/config-dir/sites-available/default (+1/-1)
To merge this branch: bzr merge lp://qastaging/~lynxman/ubuntu/oneiric/apache2/fixeddefaultsite
Reviewer Review Type Date Requested Status
James Page Disapprove
Ubuntu branches Pending
Review via email: mp+72172@code.qastaging.launchpad.net

Description of the change

  * Changed default website config (LP: #820936)
    - debian/config-dir/sites-available/default: Changed Virtualhost
    definition to avoid bug LP: #820936

To post a comment you must log in.
71. By Marc Cluet

* Expanded changelog to explain more in depth the problem we solved with this fix

Revision history for this message
James Page (james-page) wrote :

Hi Marc

As we discussed via IRC this proposed change does not address bug 820936 due to the fact that in Apache 2.2 the rewrite engine plus inheritance have to be declared explicitly in all VirtualHost definitions , see http://httpd.apache.org/docs/2.2/mod/mod_rewrite.html#vhosts.

That said the change does make some sense as it makes the default host behave in a slight nicer way with other virtualhosts - however we should try to get this change in Debian first rather that risking an Ubuntu delta which we may have to carry.

Please could you submit a bug report back to Debian.

Cheers

James

review: Disapprove

Unmerged revisions

71. By Marc Cluet

* Expanded changelog to explain more in depth the problem we solved with this fix

70. By Marc Cluet

  * Changed default website config (LP: #820936)
    - debian/config-dir/sites-available/default: Changed Virtualhost
    definition to avoid bug LP: #820936

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: