util/scripts/maintainers.go: Stub support for website tag
ME Cleaner's component has an entry specifying its website, which this parser didn't know how to handle. Avoid the resulting warning. While at it, de-C the switch statement and make it work go-style. This also fixes "R" statements being ignored. Change-Id: Ifc23e28daba9d85bf690557a80134accea8bed21 Signed-off-by: Patrick Georgi <pgeorgi@google.com> Reviewed-on: https://review.coreboot.org/29601 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org> Reviewed-by: Philipp Deppenwiese <zaolin.daisuki@gmail.com>
This commit is contained in:
		
				
					committed by
					
						
						Philipp Deppenwiese
					
				
			
			
				
	
			
			
			
						parent
						
							670cd70164
						
					
				
				
					commit
					92332635bf
				
			@@ -95,40 +95,15 @@ func build_maintainers(maintainers []string) {
 | 
			
		||||
			current.name = line
 | 
			
		||||
		} else {
 | 
			
		||||
			switch line[0] {
 | 
			
		||||
			case 'R':
 | 
			
		||||
			case 'M':
 | 
			
		||||
				{
 | 
			
		||||
			case 'R', 'M':
 | 
			
		||||
				/* Add subsystem maintainer */
 | 
			
		||||
					current.maintainer =
 | 
			
		||||
						append(current.maintainer,
 | 
			
		||||
							line[3:len(line)])
 | 
			
		||||
					break
 | 
			
		||||
				}
 | 
			
		||||
			case 'S':
 | 
			
		||||
				{
 | 
			
		||||
					break
 | 
			
		||||
				}
 | 
			
		||||
			case 'L':
 | 
			
		||||
				{
 | 
			
		||||
					break
 | 
			
		||||
				}
 | 
			
		||||
			case 'T':
 | 
			
		||||
				{
 | 
			
		||||
					break
 | 
			
		||||
				}
 | 
			
		||||
				current.maintainer = append(current.maintainer, line[3:len(line)])
 | 
			
		||||
			case 'F':
 | 
			
		||||
				{
 | 
			
		||||
				// add files
 | 
			
		||||
					current.file =
 | 
			
		||||
						append(current.file,
 | 
			
		||||
							line[3:len(line)])
 | 
			
		||||
					break
 | 
			
		||||
				}
 | 
			
		||||
				current.file = append(current.file, line[3:len(line)])
 | 
			
		||||
			case 'L', 'S', 'T', 'W': // ignore
 | 
			
		||||
			default:
 | 
			
		||||
				{
 | 
			
		||||
				fmt.Println("No such specifier: ", line)
 | 
			
		||||
					break
 | 
			
		||||
				}
 | 
			
		||||
			}
 | 
			
		||||
		}
 | 
			
		||||
	}
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user