Bug 11186 - bitbake requires vi on the host (debian jessie does not have a vi package)
Summary: bitbake requires vi on the host (debian jessie does not have a vi package)
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: core (show other bugs)
Version: 2.3
Hardware: x86 Multiple
: Undecided normal
Target Milestone: ---
Assignee: brian avery
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2017-03-17 21:57 UTC by Todor Minchev
Modified: 2017-03-22 20:32 UTC (History)
4 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Todor Minchev 2017-03-17 21:57:44 UTC 

    


    


      



        
          Comment 1
        

        
          Ross Burton

        

        
        

        
          2017-03-17 23:04:28 UTC
        

      






It does, generally installed via vim-tiny.  You must have built a fairly minimal system to not have vi present.

I can't recall what recipe needed vi, but maybe we should move it to the nonfatal list.

CCing RP as I suspect he can remember what recipe wanted vi.

    


    


      



        
          Comment 2
        

        
          brian avery

        

        
        

        
          2017-03-17 23:10:11 UTC
        

      






the cvs build which is part of the sdktest in meta/lib/oeqa/runtime/cases/buildcvs.py needs it when it configures cvs (cvs wants an editor for messages).

i have a switch to use cpio as the sdktest  which I'll send in once i can test it. then we can drop vi from the list completely.

On a separate note, we could consider moving *all* of the testimage requirements to nonfatal or even making a third category HOSTTOOLS_EXTENDED, and warn that some functionality beyond building may fail if the needed programs are not available.

    


    


      



        
          Comment 3
        

        
          brian avery

        

        
        

        
          2017-03-18 15:08:06 UTC
        

      






patch on ml to switch to cpio. Will take and close if RP ok with the switch.

    


    


      



        
          Comment 4
        

        
          brian avery

        

        
        

        
          2017-03-22 20:32:08 UTC
        

      






fixed in master: af994bfcfddb2e17d9adc13fd13870731c5ee6c7

We switched the sdk test from cvs (which needed vi) to cpio (which doesn't)