Code review comment for lp:~andreserl/maas/maas_fix_apache2_configs

Revision history for this message
Julian Edwards (julian-edwards) wrote :

41 + case $RELEASE in
42 + 12.04|12.10|13.04)

FYI I was previously told to use code names and not release names in packaging.

There is another bug you might have to fix as well - I recently upgraded from an older saucy before the apache conf change happened, and after the upgrade my apache conf was still in the wrong place.

Everything else looks ok to me, thanks for the fix!

review: Approve

« Back to merge proposal