remove duplicate tests

Author: perazz

src/fpm/manifest/package.f90

@@ -59,9 +59,6 @@ module fpm_manifest_package
 
     public :: package_config_t, new_package
 
-
-
-
     !> Package meta data
     !> Package configuration data - extends a `feature_config_t` to represent the "default" 
     !> package feature. The following are now inherited from feature_config_t: name (but for package 

src/fpm_environment.f90

@@ -82,7 +82,7 @@ pure function OS_NAME(os)
             case (OS_FREEBSD); OS_NAME =  "FreeBSD"
             case (OS_OPENBSD); OS_NAME =  "OpenBSD"
             case (OS_UNKNOWN); OS_NAME =  "Unknown"
-            case (OS_ALL)    ; OS_NAME =  "All"
+            case (OS_ALL)    ; OS_NAME =  "all"
             case default     ; OS_NAME =  "UNKNOWN"
         end select
     end function OS_NAME
@@ -113,9 +113,9 @@ pure elemental subroutine validate_os_name(os_name, is_valid)
        !> Boolean value of whether os_name is valid or not
        logical, intent(out) :: is_valid
 
-       select case (os_name)
+       select case (lower(os_name))
          case ("linux", "macos", "windows", "cygwin", "solaris", "freebsd", &
-                         & "openbsd", "unknown")
+                         & "openbsd", "all")
            is_valid = .true.
          case default
            is_valid = .false.

src/fpm_model.f90

@@ -225,7 +225,7 @@ module fpm_model
     type(string_t) :: module_prefix
 
     !> Target operating system
-    integer :: target_os = OS_UNKNOWN
+    integer :: target_os = OS_ALL
 
     contains
 
@@ -1101,8 +1101,8 @@ subroutine model_load_from_toml(self, table, error)
         self%target_os = match_os_type(os_string) 
 
     else
-        ! Default to current OS if not specified
-        self%target_os = get_os_type()
+        ! Default to ALL OS if not specified
+        self%target_os = OS_ALL
     end if
 
 end subroutine model_load_from_toml