Hi,

Is this a mistake or something intended in OSCAR/PackagePath.pm?

Please take a look at the line 873:

>     OSCAR::Logger::oscar_log_subsection "Adding $repo in $path/$distro.url";  
>                          
>     if (OSCAR::PackagePath::repos_add_urlfile ("$path/$distro.url", $repo)) { 
>                          
>         carp "ERROR: Impossible to create file $path.url";                    
>                          
>         return -1;                                                            
>                          
>     }       

The second argument of OSCAR::PackagePath::repos_add_urlfile requires an array 
but the line 873 passes a string.

This causes the distro url file not generated with the intended distro URL 
content.
(e.g., /tftpboot/distro/redhat-el-6-x86_64.url is supposed to be generated by 
"oscar-config --setup-distro redhat-el-6-x86_64" and it should have the distro 
URL which is given at /etc/oscar/supported_distros.txt but there is no distro 
URL generated at /tftpboot/distro/redhat-el-6-x86_64.url).

> [root@marvin Qt]# cat /tftpboot/distro/redhat-el-6-x86_64.url 
> # Generated file; please do not manually edit
> 


Maybe the possible solution would be generating a temporary array with $repo 
and then handing it over to repos_add_urlfile?

Regards,

--
- DongInn


------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from 
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60133471&iu=/4140/ostg.clktrk
_______________________________________________
Oscar-devel mailing list
Oscar-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/oscar-devel

Reply via email to