Hosts: ./makehosts crashing on macOS

Created on 22 May 2017  路  6Comments  路  Source: StevenBlack/hosts

On macOS I'm getting this (below) for all data.

Updating source ./data/adaway.org from https://...
Error in updating source:  https://...
Updating source ./data/add.2o7Net from https://...
Error in updating source:  https://...
etc...

Testing I find it works great at 65f0421, so commit 2ff5dc2 looks to be the cause.

bug

All 6 comments

@gfyoung ^^^

@StevenBlack : Can you replace the print statement with raise to see what the error is? I don't have access to Mac, sorry :cry:

No worries @gfyoung. It's the one commit.

The path_join_robust() thing isn't ready on macOS.

I really liked get_defaults() piece which was nice, though unrelated to path_join_robust().

As of 26d73051 both get_defaults() and path_join_robust() are extracted.

I don't yet know how path_join_robust() was failing on macOS. I just know reverting updateHostsFile.py to 65f04210c1 fixes the issue.

I'd love to see the get_defaults() piece re-submitted as a separate commit.

@StevenBlack : I actually know the error. This patch is very quick. I can resubmit everything right now.

OK!

@StevenBlack : PR is up. Could you pull down the branch to see if everything works for you now?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

scafroglia93 picture scafroglia93  路  3Comments

Laicure picture Laicure  路  3Comments

mikhoul picture mikhoul  路  3Comments

bigdargon picture bigdargon  路  3Comments

node1634 picture node1634  路  3Comments