procfs: sysfs/fixtures filenames are too long

Trying to install github.com\prometheus\client_golang\prometheus using glide which has procfs as a dependency.

The install fails on windows because sysfs/fixtures contains subfolders with long filenames:

Cloning into 'C:\Users\user\.glide\cache\src\https-github.com-prometheus-procfs'...
error: unable to create file sysfs/fixtures.src/devices/pci0000_@colon@_00/0000_@colon@_00_@colon@_0d.0/ata4/host3/target3_@colon@_0_@colon@_0/3_@colon@_0_@colon@_0_@colon@_0/block/sdb/bcache/stats_day/cache_bypass_misses: Filename too long
error: unable to create file sysfs/fixtures.src/devices/pci0000_@colon@_00/0000_@colon@_00_@colon@_0d.0/ata4/host3/target3_@colon@_0_@colon@_0/3_@colon@_0_@colon@_0_@colon@_0/block/sdb/bcache/stats_day/cache_miss_collisions: Filename too long
fatal: cannot create directory at 'sysfs/fixtures.src/devices/pci0000_@colon@_00/0000_@colon@_00_@colon@_0d.0/ata4/host3/target3_@colon@_0_@colon@_0/3_@colon@_0_@colon@_0_@colon@_0/block/sdb/bcache/stats_five_minute': Filename too long
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry the checkout with 'git checkout -f HEAD'

ref #55 #56

About this issue

Original URL
State: closed
Created 7 years ago
Comments: 23 (11 by maintainers)

Commits related to this issue

Revert "Merge pull request #56 from ideaship/copy_fixtures_02" The generated pathnames are too long for some filenames, including NTFS in its default configuration. We've considered using other separ... — committed to remijouannet/procfs by grobie 7 years ago
Merge pull request #57 from edigaryev/public-tcp-udp-readers Make TCP/UDP socket table readers public — committed to bobrik/procfs by eminence 5 years ago

Most upvoted comments

Thank you for taking the time confirm!

On Tue, Jun 20, 2017, 09:51 Francis Chuang notifications@github.com wrote:

@ideaship https://github.com/ideaship @grobie https://github.com/grobie Thanks for pushing the fix! I can confirm it works correctly! 😄

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/prometheus/procfs/issues/57#issuecomment-309673500, or mute the thread https://github.com/notifications/unsubscribe-auth/AAANaDNRyqVQrycul6Mb49VpOyXrsKd6ks5sF3oDgaJpZM4N-91g .

grobie on Jun 20, 2017

I’d say let’s go back to the tar ball. It seems to be pretty much the only way to be safe.

ideaship on Jun 20, 2017

Workaround:

Enable long paths on Windows (requires Windows 10 Anniversary Update or newer): https://superuser.com/a/1119980/97078
Configure git to use long paths: git config --global core.longpaths true (globally) or git config core.longpaths true (per project)

F21 on Jun 20, 2017