Re: Mogrify -preserve implementation
Bob Friesenhahn <[email protected]> Tue, 23 Feb 2016 08:47:58 -0600 (CST)
| Newsgroups | gmane.comp.video.graphicsmagick.core |
|---|---|
| Message-ID | <[email protected]> |
On Tue, 23 Feb 2016, Niko Rosvall wrote: > Hi, > > I wrote a fairly naive implementation to keep file timestamps when > running mogrify. I added an option -preserve to the command mogrify. > > By no means it's not perfect as it does not care about Windows. I hardly > use Windows at all, so it fairly hard for me to implement support for > it. However, I'm willing to do it, if this is a feature that GM wants to > include. Few of us love Windows, but we are dedicated to making sure that GraphicsMagick Windows users are well supported within the capabilities of the operating system. > Some questions: Would it be better to implement the actual code into > separate files (now it's just all in magick/command.c). Maybe > magic/preserve.c? What is currently typically done is to put Unix-specific code in magick/unix_port.c and Windows-specific code in magick/nt_base.c. Similar functions added to each place can allow a common implementation in magick/command.c to work. The magick/studio.h header provides a MagickFstat() macro which provides porting definitions. Windows does provide fstat as well as a a _fstati64() function (for large files). I am not sure how one sets file timestamps under Windows. Bob -- Bob Friesenhahn [email protected], http://www.simplesystems.org/users/bfriesen/ GraphicsMagick Maintainer, http://www.GraphicsMagick.org/ ------------------------------------------------------------------------------ Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140