On 12/22/2014 10:32 AM, Orion Poplawski wrote:
> On 12/22/2014 09:51 AM, Tom Eastep wrote:
>> On 12/22/2014 7:56 AM, Orion Poplawski wrote:
>>> I believe the systemd network-online.target has been around for quite a 
>>> while,
>>> so the current naming for the shorewall.service file doesn't seem correct.
>>>
>>
>> The shorewall-init.service.214 file uses the network-pre target which is
>> fairly recent. Hence the current naming convention.
>>
>> -Tom
> 
> Yeah, it makes sense for that one.  But I don't think it makes sense for:
> 
> --- shorewall-code/Shorewall/shorewall.service  2014-12-22 08:49:56.595968100
> -0700
> +++ shorewall-code/Shorewall/shorewall.service.214      2014-12-22
> 08:49:56.595968100 -0700
> @@ -5,7 +5,7 @@
>  #
>  [Unit]
>  Description=Shorewall IPv4 firewall
> -After=network.target
> +After=network-online.target
>  Conflicts=iptables.service firewalld.service
> 
>  [Service]

We didn't want to have three different names, so we bundled
network-online with pre-network under a single name. It is still easy to
install whichever you want using the tarball installers, with the
default being set in the shorewallrc file.

-Tom
-- 
Tom Eastep        \ When I die, I want to go like my Grandfather who
Shoreline,         \ died peacefully in his sleep. Not screaming like
Washington, USA     \ all of the passengers in his car
http://shorewall.net \________________________________________________

Attachment: signature.asc
Description: OpenPGP digital signature

------------------------------------------------------------------------------
Dive into the World of Parallel Programming! The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net
_______________________________________________
Shorewall-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/shorewall-devel

Reply via email to