Fix encoding discrepancy in excluded Windows paths

[ashmaroli]

Summary

When a site is in a directory containing characters with accents and similar entities (e.g., testé-çasésûr),
the paths were encoded differently which were seen as a separate directory in the site.

Context

jekyll-watch/lib/jekyll/watcher.rb

Lines 95 to 105 in adc0107

	def listen_ignore_paths(options)
	source = Pathname.new(options["source"]).expand_path
	paths = to_exclude(options)
	paths.map do |p|
	absolute_path = Pathname.new(p).expand_path
	next unless absolute_path.exist?
	begin
	relative_path = absolute_path.relative_path_from(source).to_s
	unless relative_path.start_with?("../")

If I were to have a custom destination set to "public", then the absolute_path in the code above gets created with encoding Encoding:UTF-8 while the options["source"] encoding was in
Encoding:Windows-1252.
This resulted in a "relative_path" (via Pathname) starting with ../ which caused that path to not be included as an ignored_path

#<Pathname:D:/testAc-A§asAcsA»r/public>
#<Pathname:../testAc-A§asAcsA»r/public>

or when stringified:

D:/testAc-A§asAcsA»r/public
../testAc-A§asAcsA»r/public

Steps to reproduce issue (Windows only)

• Create a site with accents in the path: jekyll new testé-çasésûr
• Add a custom destination configuration (regular string)

  destination: public

• Watch the site: bundle exec jekyll b --watch or bundle exec jekyll s
• Make a change in any file..

/cc @jekyll/windows

Thanks

To @DirtyF for providing a test repository that led to the discovery of this bug

[XhmikosR]

Shouldn't there be a test case for this?

[XhmikosR]

Oh, wait, there is. Then we should have AppVeyor set up so that this doesn't happen again.

[ashmaroli]

@XhmikosR Do you want to add the appveyor.yml..? AppVeyor has been enabled on this repo..

[XhmikosR]

I could give it a go later, but my guess is you'd handle this faster than me :P

[DirtyF]

@jekyllbot: merge +fix

[ashmaroli]

@DirtyF Before this gets shipped, we need to ensure that this does not lead to any regressions in the encoding issues patched in the current public release..