[
https://issues.apache.org/jira/browse/FLUME-1174?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13268090#comment-13268090
]
E. Sammer commented on FLUME-1174:
----------------------------------
Just to clarify a few points:
* I agree reading /etc/passwd is lame. I did that because it's guaranteed to be
available (well, obviously that's a little hyperbolic ;)) and I was really
short on time. Reading it isn't a security issue: it's world readable
(/etc/shadow is the scary one) and read by just about every tool (like ls,
id[1], etc.). The correct thing to do is write a file in a known location at
the start of the test and then read it.
* While Windows wasn't an initial target of Flume NG, I'm perfectly fine with
supporting it. We shouldn't exclude it. That said, some overwhelming number of
use cases are Unix varient based, most of which are Linux, specifically. If
people want to start opening bugs for things that don't work on Windows I'm
100% in favor of that. If you want to also contribute the fixes / patches
(including things like the necessary changes to the build / support scripts)
you're destined for committership.
* Thread.currentThread().getId() has nothing to do with the user or
/etc/passwd. It's a numeric ID of a thread.
[1] [esammer@xxx ~]$ strace -f id 2>&1 | grep /etc/passwd
open("/etc/passwd", O_RDONLY) = 3
> Attempt to build Flume v1.1.0 on Windows 7 x64 fails
> ----------------------------------------------------
>
> Key: FLUME-1174
> URL: https://issues.apache.org/jira/browse/FLUME-1174
> Project: Flume
> Issue Type: Bug
> Affects Versions: v1.1.0
> Environment: Windows 7 - 64 Bit - jdk1.7u4 - mvn 3.0.4
> Reporter: Jeff Lord
>
> It looks as if the src is expecting to be built on Linux.
> Attempting to build on Windows 7 fails with the below error that references a
> missing /etc/passwd file.
> Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.156 sec
> Running org.apache.flume.TestCounterGroup
> Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.109 sec
> Results :
> Failed tests: testRestart(org.apache.flume.source.TestExecSource)
> Tests in error:
> test(org.apache.flume.serialization.TestBodyTextEventSerializer): Unable to
> de
> lete file: src\test\resources\events.txt
> testProcess(org.apache.flume.source.TestExecSource): \etc\passwd (The
> system c
> annot find the path specified)
> Tests run: 114, Failures: 1, Errors: 2, Skipped: 2
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
