Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#56405 closed enhancement (fixed)

eccodes ENABLE_MEMFS variant

Reported by: isezen (Ismail SEZEN) Owned by: tenomoto (Takeshi Enomoto)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: ecCodes

Description

Hello,

Can you add ENABLE_MEMFS option to MacPorts ecCodes?

Thanks,

Change History (5)

comment:1 Changed 7 years ago by mf2k (Frank Schima)

Port: ecCodes added
Type: request → enhancement

In the future, please fill in the Port field.

Note that a "request" ticket type is only for requesting a new port.

comment:2 Changed 7 years ago by pmetzger (Perry E. Metzger)

  1. What is the effect of adding that option?
  2. You might get faster handling of this if you submit a Pull Request on GitHub.

comment:3 Changed 7 years ago by isezen (Ismail SEZEN)

1- It embeds definitions (normally located at /opt/local/share/eccodes/definition) in a shared library (libeccodes_memfs.so), hence if MEMFS option is ON, ecCodes read definitions from memory (if loaded before). It eliminates the need to read likely lots of files from disk.

2- I can only open an issue on GitHub. I didn't make any change (I even don't know how to) on port setup file.

comment:4 Changed 6 years ago by tenomoto (Takeshi Enomoto)

Resolution: → fixed
Status: assigned → closed

In b3ca20df03fff948eea99f7a5ddaef04c4f2980a/macports-ports (master):

ecCodes: update to 2.8.2. Add +memfs variant that includes tables in shared library

Closes: #56405

comment:5 Changed 6 years ago by tenomoto (Takeshi Enomoto)

The default value of ENABLE_MEMFS is currently OFF so I made it a variant.

Note: See TracTickets for help on using tickets.