[ros-dev] [ros-diffs] [hbelusca] 69160: [EFISYS]: Add 'bcd_hive' as a dependency. [FATTEN]: Fix help message. Functions with no parameters are void. Remove an hardcoded array size.

[Hermès BÉLUSCA-MAÏTO]

Hi (I'm answering back here for the ML history).
I added this dependency back in r69160 because at that time, efisys depended on ${CMAKE_BINARY_DIR}/boot/bootdata/BCD . But it was realized later (in r70626) that it was actually not needed; however the committer forgot to remote the bcd_hive dependency.

Best,
Hermès

-----Message d'origine-----
De : Ros-dev [mailto:ros-dev-bounces at reactos.org] De la part de Thomas Faber
Envoyé : samedi 13 mai 2017 19:20
À : Hermès BÉLUSCA-MAÏTO
Cc : ros-dev at reactos.org
Objet : Re: [ros-dev] [ros-diffs] [hbelusca] 69160: [EFISYS]: Add 'bcd_hive' as a dependency. [FATTEN]: Fix help message. Functions with no parameters are void. Remove an hardcoded array size.

Hey Hermès,
I'm fixing how the BCD hive is generated and this just showed up in an svn blame for me.
Can you explain why this bcd_hive dependency is needed? I don't see the command depending on the hive file.

Thanks,
Thomas


On 2015-09-09 21:48, hbelusca at svn.reactos.org wrote:
> Author: hbelusca
> Date: Wed Sep  9 19:48:15 2015
> New Revision: 69160
> 
> URL: http://svn.reactos.org/svn/reactos?rev=69160&view=rev
> Log:
> [EFISYS]: Add 'bcd_hive' as a dependency.
> [FATTEN]: Fix help message. Functions with no parameters are void. Remove an hardcoded array size.
> 
> Modified:
>     trunk/reactos/boot/CMakeLists.txt
>     trunk/reactos/tools/fatten/fatten.c
> 
> Modified: trunk/reactos/boot/CMakeLists.txt
> URL: 
> http://svn.reactos.org/svn/reactos/trunk/reactos/boot/CMakeLists.txt?r
> ev=69160&r1=69159&r2=69160&view=diff
> ==============================================================================
> --- trunk/reactos/boot/CMakeLists.txt	[iso-8859-1] (original)
> +++ trunk/reactos/boot/CMakeLists.txt	[iso-8859-1] Wed Sep  9 19:48:15 2015
> @@ -17,7 +17,7 @@
>  
>  add_custom_target(efisys
>      COMMAND native-fatten ${CMAKE_CURRENT_BINARY_DIR}/efisys.bin -format 2880 EFIBOOT -boot ${CMAKE_CURRENT_BINARY_DIR}/freeldr/bootsect/fat.bin -mkdir EFI -mkdir EFI/BOOT -add $<TARGET_FILE:bootmgfw> EFI/BOOT/boot${EFI_PLATFORM_ID}.efi -add ${CMAKE_BINARY_DIR}/boot/bootdata/BCD EFI/BOOT/BCD
> -    DEPENDS native-fatten bootmgfw fat
> +    DEPENDS native-fatten fat bootmgfw bcd_hive
>      VERBATIM)
>  
>  ##bootcd
> 

_______________________________________________
Ros-dev mailing list
Ros-dev at reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev