Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • P Pacman
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Graph
    • Compare
    • Locked Files
  • Issues 8
    • Issues 8
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 21
    • Merge requests 21
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Releases
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Pacman
  • Pacman
  • Repository
Switch branch/tag
  • pacman
  • scripts
  • makepkg.sh.in
Find file BlameHistoryPermalink
  • Eli Schwartz's avatar
    makepkg: simplify run_pacman logic · 656e851f
    Eli Schwartz authored Feb 20, 2019 and Allan McRae's avatar Allan McRae committed Feb 21, 2019
    We don't need to check the options twice, since it is the same check
    both times. Instead, merge the conditionals.
    
    As far as I can tell, the only reason the checks for:
    
    - PACMAN_OPTS and
    - whether to use sudo
    
    were ever separated is due to the historic existence of --asroot, since
    the second check included a check for (( ! ASROOT )) until it was
    cleaned up in commit 61ba5c96
    
    .
    
    Signed-off-by: Eli Schwartz's avatarEli Schwartz <eschwartz@archlinux.org>
    Signed-off-by: Allan McRae's avatarAllan McRae <allan@archlinux.org>
    656e851f