public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "gaoliming" <gaoliming@byosoft.com.cn>
To: "'Andrew Fish'" <afish@apple.com>
Cc: <devel@edk2.groups.io>
Subject: 回复: [edk2-devel] Why is MAX_LONG_FILE_PATH 500 vs. PATH_MAX?
Date: Fri, 30 Oct 2020 11:10:47 +0800	[thread overview]
Message-ID: <008301d6ae6a$43c3f350$cb4bd9f0$@byosoft.com.cn> (raw)
In-Reply-To: <93CD311F-B4D9-4AF9-8AD4-3093E04FBB0F@apple.com>

[-- Attachment #1: Type: text/plain, Size: 3671 bytes --]

No. It doesn’t work on Windows. So, need to include it on non Windows OS. 

 

#ifndef _WIN32

#include <limits.h>

#endif

 

Thanks

Liming

发件人: Andrew Fish <afish@apple.com> 
发送时间: 2020年10月30日 10:55
收件人: gaoliming <gaoliming@byosoft.com.cn>
抄送: devel@edk2.groups.io
主题: Re: [edk2-devel] Why is MAX_LONG_FILE_PATH 500 vs. PATH_MAX?

 

Does that work on Windows? 

 

Thanks,

 

Andrew Fish





On Oct 29, 2020, at 7:53 PM, gaoliming <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> > wrote:

 

Yes. #include <limits.h> is required for PATH_MAX

https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/limits.h.html

 

 

发件人: bounce+27952+66797+4905953+8761045@groups.io <mailto:bounce+27952+66797+4905953+8761045@groups.io>  <bounce+27952+66797+4905953+8761045@groups.io <mailto:bounce+27952+66797+4905953+8761045@groups.io> > 代表 Andrew Fish via groups.io <http://groups.io/> 
发送时间: 2020年10月30日 10:28
收件人: devel@edk2.groups.io <mailto:devel@edk2.groups.io> ; gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> 
主题: Re: [edk2-devel] Why is MAX_LONG_FILE_PATH 500 vs. PATH_MAX?

 

 






On Oct 29, 2020, at 7:25 PM, gaoliming <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> > wrote:

 

Andrew:

 Windows defines _MAX_PATH macro. But, its value is 260. It is not enough. To fix your issue, MAX_LONG_FILE_PATH can be updated as below. 

 

 

OK I’ll file a BZ. It may also require adding an include?

 

Thanks,

 

Andrew Fish






#ifdef PATH_MAX

#define MAX_LONG_FILE_PATH PATH_MAX

#else

#define MAX_LONG_FILE_PATH 500

#endif

 

Thanks

Liming

发件人: bounce+27952+66793+4905953+8761045@groups.io <mailto:bounce+27952+66793+4905953+8761045@groups.io>  <bounce+27952+66793+4905953+8761045@groups.io <mailto:bounce+27952+66793+4905953+8761045@groups.io> > 代表 Andrew Fish via groups.io <http://groups.io/> 
发送时间: 2020年10月30日 9:37
收件人: gaoliming <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> >
抄送: devel@edk2.groups.io <mailto:devel@edk2.groups.io> 
主题: Re: [edk2-devel] Why is MAX_LONG_FILE_PATH 500 vs. PATH_MAX?

 

 







On Oct 29, 2020, at 6:34 PM, gaoliming <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> > wrote:

 

This macro is added to resolve the limitation on WINDOWS OS for the length of entire file path can’t be larger than 255. It use the hard code 500, because 500 is enough for most case. Do you meet with the problem that the file path length exceeds 500?

 

 

Yes locally we have changed it to PATH_MAX to fix a build break we hit. Does Windows support PATH_MAX?

 

Thanks,

 

Andrew Fish







Thanks

Liming

发件人: bounce+27952+66762+4905953+8761045@groups.io <mailto:bounce+27952+66762+4905953+8761045@groups.io>  <bounce+27952+66762+4905953+8761045@groups.io <mailto:bounce+27952+66762+4905953+8761045@groups.io> > 代表 Andrew Fish via groups.io <http://groups.io/> 
发送时间: 2020年10月30日 1:24
收件人: edk2-devel-groups-io <devel@edk2.groups.io <mailto:devel@edk2.groups.io> >
主题: [edk2-devel] Why is MAX_LONG_FILE_PATH 500 vs. PATH_MAX?

 

Does anyone know why MAX_LONG_FILE_PATH [1] is hard coded to 500 vs. using the systems PATH_MAX?

 

On Linux I think PATH_MAX is 4KiB and on macOS it is 1KiB.

 

 

[1] https://github.com/tianocore/edk2/blob/master/BaseTools/Source/C/Common/CommonLib.h#L17

 

Thanks,

 

Andrew Fish

 

 



 


[-- Attachment #2: Type: text/html, Size: 20912 bytes --]

      reply	other threads:[~2020-10-30  3:10 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-29 17:24 Why is MAX_LONG_FILE_PATH 500 vs. PATH_MAX? Andrew Fish
2020-10-30  1:34 ` 回复: [edk2-devel] " gaoliming
2020-10-30  1:37   ` Andrew Fish
2020-10-30  2:25     ` 回复: " gaoliming
2020-10-30  2:27       ` Andrew Fish
2020-10-30  2:53         ` 回复: " gaoliming
2020-10-30  2:54           ` Andrew Fish
2020-10-30  3:10             ` gaoliming [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='008301d6ae6a$43c3f350$cb4bd9f0$@byosoft.com.cn' \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox