zarb-ml/mageia-sysadm/2010-November/000808.html


1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
 <HEAD>
   <TITLE> [Mageia-sysadm] [412] - it is IMHO cleaner and logical to have apache:: vhost_other_app directly requiring apache:: base rather than letting every people adding it when using the class
   </TITLE>
   <LINK REL="Index" HREF="index.html" >
   <LINK REL="made" HREF="mailto:mageia-sysadm%40mageia.org?Subject=Re%3A%20%5BMageia-sysadm%5D%20%5B412%5D%20-%20it%20is%20IMHO%20cleaner%20and%20logical%20to%20have%0A%20apache%3A%3A%20vhost_other_app%20directly%20requiring%20apache%3A%3A%20base%20rather%20than%0A%20letting%20every%20people%20adding%20it%20when%20using%20the%20class&In-Reply-To=%3C20101123011405.F26EC3FC92%40valstar.mageia.org%3E">
   <META NAME="robots" CONTENT="index,nofollow">
   <META http-equiv="Content-Type" content="text/html; charset=us-ascii">
   <LINK REL="Previous"  HREF="000807.html">
   <LINK REL="Next"  HREF="000817.html">
 </HEAD>
 <BODY BGCOLOR="#ffffff">
   <H1>[Mageia-sysadm] [412] - it is IMHO cleaner and logical to have apache:: vhost_other_app directly requiring apache:: base rather than letting every people adding it when using the class</H1>
    <B>root at mageia.org</B> 
    <A HREF="mailto:mageia-sysadm%40mageia.org?Subject=Re%3A%20%5BMageia-sysadm%5D%20%5B412%5D%20-%20it%20is%20IMHO%20cleaner%20and%20logical%20to%20have%0A%20apache%3A%3A%20vhost_other_app%20directly%20requiring%20apache%3A%3A%20base%20rather%20than%0A%20letting%20every%20people%20adding%20it%20when%20using%20the%20class&In-Reply-To=%3C20101123011405.F26EC3FC92%40valstar.mageia.org%3E"
       TITLE="[Mageia-sysadm] [412] - it is IMHO cleaner and logical to have apache:: vhost_other_app directly requiring apache:: base rather than letting every people adding it when using the class">root at mageia.org
       </A><BR>
    <I>Tue Nov 23 02:14:05 CET 2010</I>
    <P><UL>
        <LI>Previous message: <A HREF="000807.html">[Mageia-sysadm] [411] - add a comment so I do not have the	impression to have
</A></li>
        <LI>Next message: <A HREF="000817.html">[Mageia-sysadm] [412] - it is IMHO cleaner and logical to have	apache:: vhost_other_app directly requiring apache:: base	rather than letting every people adding it when using the class
</A></li>
         <LI> <B>Messages sorted by:</B> 
              <a href="date.html#808">[ date ]</a>
              <a href="thread.html#808">[ thread ]</a>
              <a href="subject.html#808">[ subject ]</a>
              <a href="author.html#808">[ author ]</a>
         </LI>
       </UL>
    <HR>  
<!--beginarticle-->
<PRE>Revision: 412
Author:   misc
Date:     2010-11-23 02:14:05 +0100 (Tue, 23 Nov 2010)
Log Message:
-----------
- it is IMHO cleaner and logical to have apache::vhost_other_app directly requiring apache::base rather than letting every people adding it when using the class

Modified Paths:
--------------
    puppet/modules/apache/manifests/init.pp
    puppet/modules/buildsystem/manifests/init.pp

Modified: puppet/modules/apache/manifests/init.pp
===================================================================
--- puppet/modules/apache/manifests/init.pp	2010-11-23 01:11:15 UTC (rev 411)
+++ puppet/modules/apache/manifests/init.pp	2010-11-23 01:14:05 UTC (rev 412)
@@ -130,6 +130,7 @@
     }
 
    define vhost_other_app($vhost_file) {
+        include apache::base
         file { &quot;$name.conf&quot;:
             path =&gt; &quot;/etc/httpd/conf/vhosts.d/$name.conf&quot;,
             ensure =&gt; &quot;present&quot;,

Modified: puppet/modules/buildsystem/manifests/init.pp
===================================================================
--- puppet/modules/buildsystem/manifests/init.pp	2010-11-23 01:11:15 UTC (rev 411)
+++ puppet/modules/buildsystem/manifests/init.pp	2010-11-23 01:14:05 UTC (rev 412)
@@ -27,7 +27,6 @@
             ensure =&gt; &quot;installed&quot;
         }
 
-        include apache::base
         apache::vhost_other_app { &quot;repository.$domain&quot;:
             vhost_file =&gt; &quot;buildsystem/vhost_repository.conf&quot;,
         }
-------------- next part --------------
An HTML attachment was scrubbed...
URL: &lt;/pipermail/mageia-sysadm/attachments/20101123/815cc2b7/attachment-0001.html&gt;
</PRE>


<!--endarticle-->
    <HR>
    <P><UL>
        <!--threads-->
	<LI>Previous message: <A HREF="000807.html">[Mageia-sysadm] [411] - add a comment so I do not have the	impression to have
</A></li>
	<LI>Next message: <A HREF="000817.html">[Mageia-sysadm] [412] - it is IMHO cleaner and logical to have	apache:: vhost_other_app directly requiring apache:: base	rather than letting every people adding it when using the class
</A></li>
         <LI> <B>Messages sorted by:</B> 
              <a href="date.html#808">[ date ]</a>
              <a href="thread.html#808">[ thread ]</a>
              <a href="subject.html#808">[ subject ]</a>
              <a href="author.html#808">[ author ]</a>
         </LI>
       </UL>

<hr>
<a href="https://www.mageia.org/mailman/listinfo/mageia-sysadm">More information about the Mageia-sysadm
mailing list</a><br>
</body></html>