Update: package%build_config not allocatable

Author: LKedward

fpm/src/fpm/manifest/package.f90

@@ -49,7 +49,7 @@ module fpm_manifest_package
         character(len=:), allocatable :: name
 
         !> Build configuration data
-        type(build_config_t), allocatable :: build_config
+        type(build_config_t) :: build_config
 
         !> Library meta data
         type(library_t), allocatable :: library
@@ -107,7 +107,6 @@ subroutine new_package(self, table, error)
             call fatal_error(error, "Type mismatch for build entry, must be a table")
             return
         end if
-        allocate(self%build_config)
         call new_build_config(self%build_config, child, error)
         if (allocated(error)) return
 
@@ -242,9 +241,7 @@ subroutine info(self, unit, verbosity)
             write(unit, fmt) "- name", self%name
         end if
 
-        if (allocated(self%build_config)) then
-            call self%build_config%info(unit, pr - 1)
-        end if
+        call self%build_config%info(unit, pr - 1)
 
         if (allocated(self%library)) then
             write(unit, fmt) "- target", "archive"

fpm/test/fpm_test/test_manifest.f90

@@ -101,11 +101,6 @@ subroutine test_valid_manifest(error)
             return
         end if
 
-        if (.not.allocated(package%build_config)) then
-            call test_failed(error, "build is not present in package data")
-            return
-        end if
-
         if (.not.allocated(package%library)) then
             call test_failed(error, "library is not present in package data")
             return
@@ -482,11 +477,6 @@ subroutine test_build_config_valid(error)
 
         if (allocated(error)) return
 
-        if (.not.allocated(package%build_config)) then
-            call test_failed(error, "build is not present in package data")
-            return
-        end if
-
         if (package%build_config%auto_executables) then
             call test_failed(error, "Wong value of 'auto-executables' read, expecting .false.")
             return
@@ -523,11 +513,6 @@ subroutine test_build_config_empty(error)
 
         if (allocated(error)) return
 
-        if (.not.allocated(package%build_config)) then
-            call test_failed(error, "build is not present in package data")
-            return
-        end if
-
         if (.not.package%build_config%auto_executables) then
             call test_failed(error, "Wong default value of 'auto-executables' read, expecting .true.")
             return