Fix c++11 0.27

CMakeLists.txt

@@ -9,7 +9,6 @@ project(exiv2          # use TWEAK to categorize the build
                        # 0.27.3.19 = RC1 Not for release
     LANGUAGES CXX C
 )
-
 include(cmake/mainSetup.cmake  REQUIRED)
 
 # options and their default values
@@ -29,8 +28,9 @@ option( EXIV2_ENABLE_SSH              "USE Libssh for SshIo (WEBREADY)"
 
 option( EXIV2_BUILD_SAMPLES           "Build sample applications"                             ON  )
 option( EXIV2_BUILD_EXIV2_COMMAND     "Build exiv2 command-line executable"                   ON  )
-option( EXIV2_BUILD_UNIT_TESTS        "Build unit tests"                                      OFF )
+option( EXIV2_BUILD_UNIT_TESTS        "Build unit tests"                                      ON  )
 option( EXIV2_BUILD_DOC               "Add 'doc' target to generate documentation"            OFF )
+option( EXIV2_BUILD_USE_C++11         "Use the C++11 compiler"                                OFF )
 
 # Only intended to be used by Exiv2 developers/contributors
 option( EXIV2_TEAM_EXTRA_WARNINGS     "Add more sanity checks using compiler flags"           OFF )

README-SAMPLES.md

@@ -195,7 +195,7 @@ $
 #### geotag
 
 ```
-Usage: geotag {-help|-version|-dst|-dryrun|-ascii|-verbose|-adjust value|-tz value|-delta value}+ path+
+Usage: geotag {-help|-version|-dst|-dryrun|-ascii|-remove|-verbose|-adjust value|-tz value|-delta value}+ path+
 ```
 
 Geotag reads one or more GPX files and adds GPS Tages to images.  _Code: [geotag.cpp](samples/geotag.cpp)_
@@ -208,6 +208,7 @@ If the path is a directory, geotag will read all the files in the directory.  It
 | -dst      | Apply 1 hour adjustment for daylight saving time. |
 | -dryrun   | Read arguments and print report.  Does not modify images. |
 | -verbose  | Report progress. |
+| -remove   | Removes GPS tags from images. |
 | -adjust  value | Add/subtract time from image data. |
 | -tz      value | Specify time zone. For example PST = -8:00 |
 | -delta   value | Correction between Image DataTime and GPS time. |
@@ -643,4 +644,4 @@ Read an XMP packet from a file, parse and re-serialize it.
 
 Robin Mills<br>
 [email protected]<br>
-Revised: 2019-06-20
+Revised: 2020-05-01

README.md

@@ -23,6 +23,7 @@
    14. [Thread Safety](#2-14)
    15. [Library Initialisation and Cleanup](#2-15)
    16. [Cross Platform Build and Test on Linux for MinGW](#2-16)
+   17. [Building with C++11 and other compilers](#2-17)
 3. [License and Support](#3)
     1. [License](#3-1)
     2. [Support](#3-2)
@@ -696,6 +697,24 @@ You will find that 3 tests fail at the end of the test suite.  It is safe to ign
 
 [TOC](#TOC)
 
+<div id="2-17">
+
+### 2.17 Building with C++11 and other compilers
+
+Exiv2 uses the default compiler for your system.  Exiv2 v0.27 was written to the C++ 1998 standard and will compile with C++11 and C++14.  As auto_ptr support is no longer in C++17, you cannot build with that system.  Exiv2 v0.28 and later do not use auto_ptr and are fully supported with all compilers.
+
+To generate a build with C++11:
+
+```bash
+$ cd <exiv2dir>
+$ mkdir build ; cd build
+$ cmake .. -DCMAKE_CXX_STANDARD=11 -DCMAKE_CXX_FLAGS=-Wno-deprecated
+```
+
+The option -DCMAKE\_CXX\_FLAGS=-Wno-deprecated suppresses warnings from C++11 concerning auto_ptr which is deprecated.
+
+[TOC](#TOC)
+
 <div id="3">
 
 ## 3 License and Support
@@ -1167,4 +1186,4 @@ $ sudo pkg install developer/gcc-7
 
 [TOC](#TOC)
 
-Written by Robin Mills<br>[email protected]<br>Updated: 2020-04-21
+Written by Robin Mills<br>[email protected]<br>Updated: 2020-05-02

include/exiv2/config.h

@@ -93,5 +93,15 @@ typedef int pid_t;
 #endif
 //////////////////////////////////////
 
+// https://softwareengineering.stackexchange.com/questions/291141/how-to-handle-design-changes-for-auto-ptr-deprecation-in-c11
+#include <memory>
+#if __cplusplus >= 201103L
+  #include <sys/types.h>
+  #include <unistd.h>
+  template <typename T>
+  using auto_ptr = std::unique_ptr<T>;
+#else
+  using std::auto_ptr;
+#endif
 
 #endif // _CONFIG_H_

samples/CMakeLists.txt

@@ -120,7 +120,6 @@ list(APPEND APPLICATIONS remotetest)
 # ******************************************************************************
 foreach(application ${APPLICATIONS})
     target_link_libraries(${application} PRIVATE exiv2lib)
-    target_compile_features(${application} PRIVATE cxx_std_98)
     if( EXIV2_ENABLE_PNG )
         target_link_libraries( ${application} PRIVATE ${ZLIB_LIBRARIES} )
     endif()

samples/geotag.cpp

@@ -87,6 +87,7 @@ class Options  {
     bool        dst;
     bool        dryrun;
     bool        ascii;
+    bool        remove;
 
     Options()
     {
@@ -96,6 +97,7 @@ class Options  {
         dst         = false;
         dryrun      = false;
         ascii       = false;
+        remove      = false;
     }
 
     virtual ~Options() {} ;
@@ -113,6 +115,7 @@ enum                        // keyword indices
 ,   kwDST
 ,   kwDRYRUN
 ,   kwASCII
+,   kwREMOVE
 ,   kwVERBOSE
 ,   kwADJUST
 ,   kwTZ
@@ -790,6 +793,7 @@ int main(int argc,const char* argv[])
     keywords[kwADJUST  ] = "adjust";
     keywords[kwTZ      ] = "tz";
     keywords[kwDELTA   ] = "delta";
+    keywords[kwREMOVE  ] = "remove";
 
     map<std::string,string> shorts;
     shorts["-?"] = "-help";
@@ -803,6 +807,7 @@ int main(int argc,const char* argv[])
     shorts["-s"] = "-delta";
     shorts["-X"] = "-dryrun";
     shorts["-a"] = "-ascii";
+    shorts["-r"] = "-remove";
 
     Options options ;
     options.help    = sina(keywords[kwHELP   ],argv) || argc < 2;
@@ -811,6 +816,7 @@ int main(int argc,const char* argv[])
     options.version = sina(keywords[kwVERSION],argv);
     options.dst     = sina(keywords[kwDST    ],argv);
     options.ascii   = sina(keywords[kwASCII  ],argv);
+    options.remove  = sina(keywords[kwASCII  ],argv);
 
     for ( int i = 1 ; !result && i < argc ; i++ ) {
         const char* arg   = argv[i++];
@@ -831,6 +837,7 @@ int main(int argc,const char* argv[])
             case kwDRYRUN   : options.dryrun      = true ; break;
             case kwVERBOSE  : options.verbose     = true ; break;
             case kwASCII    : options.ascii       = true ; break;
+            case kwREMOVE   : options.remove      = true ; break;
             case kwTZ       : Position::tz_       = parseTZ(value);break;
             case kwADJUST   : Position::adjust_   = ivalue;break;
             case kwDELTA    : Position::deltaMax_ = ivalue;break;
@@ -881,8 +888,8 @@ int main(int argc,const char* argv[])
                 s-= m*60  ;
             printf("tz,dst,adjust = %d,%d,%d total = %dsecs (= %d:%d:%d)\n",t,d,a,A,h,m,s);
         }
-/*
-        if ( options.verbose ) {
+
+        if ( options.verbose && gTimeDict.size() ) {
             printf("Time Dictionary\n");
             for ( TimeDict_i it = gTimeDict.begin() ;  it != gTimeDict.end() ; it++ ) {
                 std::string sTime = getExifTime(it->first);
@@ -894,7 +901,10 @@ int main(int argc,const char* argv[])
                 printf("%s %s\n",sTime.c_str(), sPos.c_str());
             }
         }
-*/
+
+        if ( !gTimeDict.size() && gFiles.size() && !options.remove  ) {
+            printf("nothing in time dictionary!\n");
+        }
         for ( size_t p = 0 ; p < gFiles.size() ; p++ ) {
             std::string path  = gFiles[p] ;
             std::string stamp ;
@@ -905,8 +915,22 @@ int main(int argc,const char* argv[])
                 if ( image.get() ) {
                     image->readMetadata();
                     Exiv2::ExifData& exifData = image->exifData();
+
+                    // if asked to remove, cycle the metadata and remove GPSInfo
+                    // you can remove and geotag in the same run
+                    // this is very useful when the TZ is incorrectly set to "clean" GPS before writing new data
+                    if ( options.remove ) {
+                        for (Exiv2::ExifData::iterator pos = exifData.begin(); pos != exifData.end(); ++pos) {
+                            if ( pos->groupName() == "GPSInfo" ) {
+                                if ( options.verbose ) {
+                                    std::cout << "remove: " << pos->key() << std::endl;
+                                }
+                                if ( !options.dryrun ) exifData.erase(pos);
+                            }
+                        }
+                    } 
                     if ( pPos ) {
-                        exifData["Exif.GPSInfo.GPSProcessingMethod" ] = "65 83 67 73 73 0 0 0 72 89 66 82 73 68 45 70 73 88"; // ASCII HYBRID-FIX
+                        exifData["Exif.GPSInfo.GPSProcessingMethod" ] = "charset=Ascii HYBRID-FIX";
                         exifData["Exif.GPSInfo.GPSVersionID"        ] = "2 2 0 0";
                         exifData["Exif.GPSInfo.GPSMapDatum"         ] = "WGS-84";
 
@@ -924,7 +948,9 @@ int main(int argc,const char* argv[])
 
                         printf("%s %s % 2d\n",path.c_str(),pPos->toString().c_str(),pPos->delta());
                     } else {
-                        printf("%s *** not in time dict ***\n",path.c_str());
+                        if ( gTimeDict.size() ) {
+                            printf("%s *** not in time dict ***\n",path.c_str());
+                        }
                     }
                     if ( !options.dryrun ) image->writeMetadata();
                 }

src/CMakeLists.txt

@@ -91,10 +91,6 @@ generate_export_header(exiv2lib
     STATIC_DEFINE exiv2lib_STATIC
 )
 
-target_compile_features(exiv2lib_int PRIVATE cxx_std_98)
-target_compile_features(exiv2lib PRIVATE cxx_std_98)
-
-
 # Conditional addition of sources to library targets
 # ---------------------------------------------------------
 
@@ -256,7 +252,6 @@ if(EXIV2_BUILD_EXIV2_COMMAND)
          getopt.cpp getopt.hpp
          utils.cpp utils.hpp
     )
-    target_compile_features(exiv2 PRIVATE cxx_std_98)
 
     set_target_properties( exiv2 PROPERTIES
         COMPILE_FLAGS ${EXTRA_COMPILE_FLAGS}

src/basicio.cpp

@@ -80,7 +80,6 @@ typedef short nlink_t;
 // *****************************************************************************
 // class member definitions
 namespace Exiv2 {
-
     BasicIo::~BasicIo()
     {
     }

test/data/geotag-test.out

@@ -23,5 +23,5 @@ Exif.GPSInfo.GPSAltitudeRef                  Byte        1  Below sea level
 Exif.GPSInfo.GPSAltitude                     Rational    1  14.3 m
 Exif.GPSInfo.GPSTimeStamp                    Rational    3  09:54:28
 Exif.GPSInfo.GPSMapDatum                     Ascii       7  WGS-84
-Exif.GPSInfo.GPSProcessingMethod             Undefined  58  65 83 67 73 73 0 0 0 72 89 66 82 73 68 45 70 73 88
+Exif.GPSInfo.GPSProcessingMethod             Undefined  18  HYBRID-FIX
 Exif.GPSInfo.GPSDateStamp                    Ascii      20  2008:05:08 09:54:28

unitTests/CMakeLists.txt

@@ -42,7 +42,6 @@ target_compile_definitions(unit_tests
     exiv2lib_STATIC
     TESTDATA_PATH="${PROJECT_SOURCE_DIR}/test/data"
 )
-target_compile_features(unit_tests PRIVATE cxx_std_98)
 
 if (exiv2lib_COMPILE_DEFINITIONS)
     target_compile_definitions(unit_tests PRIVATE ${exiv2lib_COMPILE_DEFINITIONS})