All of lore.kernel.org
 help / color / mirror / Atom feed
From: Randy MacLeod <randy.macleod@windriver.com>
To: <jackie.huang@windriver.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 0/1] remove the unnecessary protocol parameters
Date: Fri, 30 Aug 2013 00:27:41 -0400	[thread overview]
Message-ID: <52201F3D.7020607@windriver.com> (raw)
In-Reply-To: <cover.1377827133.git.jackie.huang@windriver.com>

On 13-08-29 09:56 PM, jackie.huang@windriver.com wrote:
> From: Jackie Huang <jackie.huang@windriver.com>
>
> What I did to remove these:
> I checked that we only use git and svn:
> $ find meta \( -name *.bb -o -name *.inc -o -name *.bbclass \) -exec sed -n 's/.*"\(\w*:\/\/\).*\(;protocol=\w*\)/\1 \2/p' {} \; | sort | uniq
> git:// ;protocol=git \
> git:// ;protocol=git"
> git:// ;protocol=git;bareclone=1;branch=${KBRANCH},${KMETA};name=machine,meta"
> git:// ;protocol=git;bareclone=1;branch=${KBRANCH},meta;name=machine,meta"
> git:// ;protocol=git" % (gstpkg_basename, branch))
> git:// ;protocol=git;nocheckout=1;branch=${KBRANCH},${KMETA};name=machine,meta"
> git:// ;protocol=http \
> svn:// ;protocol=http \
> svn:// ;protocol=http;module=trunk \
> svn:// ;protocol=svn"
>
> So I think it's safe to simply remove ";protocol=git" and ";protocol=svn":
>
> $ find meta \( -name *.bb -o -name *.inc -o -name *.bbclass \) -exec sed -i 's/;protocol=git//; s/;protocol=svn//' {} \;
>
> What left with protocol parameter after removal is:
> $ find meta \( -name *.bb -o -name *.inc -o -name *.bbclass \) -exec sed -n 's/.*"\(\w*:\/\/\).*\(;protocol=\w*\)/\1 \2/p' {} \; | sort | uniq
> git:// ;protocol=http \
> svn:// ;protocol=http \
> svn:// ;protocol=http;module=trunk
>
> Please refer to "[OE-core] [PATCH] u-boot: update to 2013.07" for detail discussions.
>
> tested info:
> MACHINE ??= "qemux86-64"
>
> $ bitbake world
>
> --
> The following changes since commit 7d93288117e7054472d8a01dde0b38bc0ff98c27:
>
>    buildtools-tarball: Add python-pkgutil (2013-08-29 00:22:49 +0100)
>
> are available in the git repository at:
>    git://git.pokylinux.org/poky-contrib jhuang0/d_remove_protocol_0830_0
>    http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=jhuang0/d_remove_protocol_0830_0
>
> Jackie Huang (1):
>    remove the unnecessary protocol parameters
>
>   meta/classes/clutter.bbclass                       |    2 +-
>   .../eee-acpi-scripts/eee-acpi-scripts_git.bb       |    2 +-
>   .../u-boot/u-boot-fw-utils_2013.01.01.bb           |    2 +-
>   .../u-boot/u-boot-mkimage_2013.01.01.bb            |    2 +-
>   meta/recipes-bsp/u-boot/u-boot_2011.03.bb          |    2 +-
>   meta/recipes-bsp/u-boot/u-boot_2011.06.bb          |    2 +-
>   meta/recipes-bsp/u-boot/u-boot_2013.01.01.bb       |    2 +-
>   meta/recipes-bsp/u-boot/u-boot_2013.07.bb          |    2 +-
>   meta/recipes-bsp/x-load/x-load_git.bb              |    6 +++---
>   meta/recipes-bsp/zaurusd/zaurusd_git.bb            |    2 +-
>   .../connman/connman-gnome_0.7.bb                   |    2 +-
>   .../mobile-broadband-provider-info_git.bb          |    2 +-
>   meta/recipes-connectivity/neard/neard_0.10.bb      |    2 +-
>   meta/recipes-connectivity/ofono/ofono_git.bb       |    2 +-
>   meta/recipes-core/dbus-wait/dbus-wait_git.bb       |    2 +-
>   .../images/build-appliance-image_8.0.bb            |    2 +-
>   meta/recipes-core/psplash/psplash_git.bb           |    2 +-
>   meta/recipes-core/uclibc/uclibc-git.inc            |    2 +-
>   meta/recipes-core/update-rc.d/update-rc.d_0.7.bb   |    2 +-
>   .../btrfs-tools/btrfs-tools_git.bb                 |    2 +-
>   meta/recipes-devtools/gnu-config/gnu-config_git.bb |    2 +-
>   meta/recipes-devtools/mtd/mtd-utils_1.5.0.bb       |    2 +-
>   meta/recipes-devtools/opkg-utils/opkg-utils_git.bb |    2 +-
>   meta/recipes-devtools/pkgconfig/pkgconfig_git.bb   |    2 +-
>   meta/recipes-devtools/prelink/prelink_git.bb       |    2 +-
>   meta/recipes-devtools/pseudo/pseudo_git.bb         |    2 +-
>   meta/recipes-devtools/qemu/qemu_git.bb             |    2 +-
>   .../recipes-devtools/swabber/swabber-native_git.bb |    2 +-
>   meta/recipes-devtools/tcf-agent/tcf-agent_git.bb   |    2 +-
>   .../recipes-gnome/gtk-doc-stub/gtk-doc-stub_git.bb |    2 +-
>   .../gtk-theme-torturer/gtk-theme-torturer_git.bb   |    2 +-
>   meta/recipes-graphics/drm/libdrm_git.bb            |    2 +-
>   meta/recipes-graphics/fstests/fstests_git.bb       |    2 +-
>   meta/recipes-graphics/libfakekey/libfakekey_git.bb |    2 +-
>   .../matchbox-wm/matchbox-wm_git.bb                 |    2 +-
>   meta/recipes-graphics/mesa/mesa_git.bb             |    2 +-
>   meta/recipes-graphics/mx/mx-1.0_1.4.7.bb           |    2 +-
>   meta/recipes-graphics/xcb/libxcb_git.bb            |    2 +-
>   meta/recipes-graphics/xcb/xcb-proto_git.bb         |    2 +-
>   .../xinput-calibrator/xinput-calibrator_git.bb     |    2 +-
>   .../xorg-driver/xf86-input-keyboard_git.bb         |    2 +-
>   .../xorg-driver/xf86-input-mouse_git.bb            |    2 +-
>   .../xorg-driver/xf86-input-synaptics_git.bb        |    2 +-
>   .../xorg-driver/xf86-video-intel_git.bb            |    2 +-
>   .../xorg-driver/xf86-video-omap_git.bb             |    2 +-
>   .../recipes-graphics/xorg-lib/libxcalibrate_git.bb |    2 +-
>   meta/recipes-graphics/xorg-lib/libxext_git.bb      |    2 +-
>   meta/recipes-graphics/xorg-lib/libxi_git.bb        |    2 +-
>   .../xorg-proto/calibrateproto_git.bb               |    2 +-
>   meta/recipes-graphics/xorg-proto/dri2proto_git.bb  |    2 +-
>   meta/recipes-graphics/xorg-proto/inputproto_git.bb |    2 +-
>   .../xvideo-tests/xvideo-tests_git.bb               |    2 +-
>   meta/recipes-kernel/blktrace/blktrace_git.bb       |    2 +-
>   meta/recipes-kernel/dtc/dtc.inc                    |    2 +-
>   .../kern-tools/kern-tools-native_git.bb            |    2 +-
>   .../linux-firmware/linux-firmware_git.bb           |    2 +-
>   meta/recipes-kernel/linux/linux-yocto-dev.bb       |    2 +-
>   meta/recipes-kernel/linux/linux-yocto-rt_3.10.bb   |    2 +-
>   meta/recipes-kernel/linux/linux-yocto-rt_3.4.bb    |    2 +-
>   meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb    |    2 +-
>   meta/recipes-kernel/linux/linux-yocto-tiny_3.10.bb |    2 +-
>   meta/recipes-kernel/linux/linux-yocto-tiny_3.4.bb  |    2 +-
>   meta/recipes-kernel/linux/linux-yocto-tiny_3.8.bb  |    2 +-
>   meta/recipes-kernel/linux/linux-yocto_3.10.bb      |    2 +-
>   meta/recipes-kernel/linux/linux-yocto_3.4.bb       |    2 +-
>   meta/recipes-kernel/linux/linux-yocto_3.8.bb       |    2 +-
>   meta/recipes-kernel/lttng/babeltrace_1.1.1.bb      |    2 +-
>   meta/recipes-kernel/lttng/lttng-modules_git.bb     |    2 +-
>   meta/recipes-kernel/lttng/lttng-tools_2.2.0.bb     |    2 +-
>   meta/recipes-kernel/lttng/lttng-ust_2.2.0.bb       |    2 +-
>   meta/recipes-kernel/oprofile/oprofile_git.bb       |    2 +-
>   .../oprofile/oprofileui-server_git.bb              |    2 +-
>   meta/recipes-kernel/oprofile/oprofileui_git.bb     |    2 +-
>   meta/recipes-kernel/sysprof/sysprof_git.bb         |    2 +-
>   meta/recipes-kernel/systemtap/systemtap_git.inc    |    2 +-
>   meta/recipes-kernel/trace-cmd/kernelshark_1.2.bb   |    2 +-
>   meta/recipes-kernel/trace-cmd/trace-cmd_1.2.bb     |    2 +-
>   .../gstreamer/gstreamer1.0-libav_git.bb            |    2 +-
>   .../gstreamer/gstreamer1.0-omx_git.bb              |    2 +-
>   .../gstreamer/gstreamer1.0-plugins.inc             |    2 +-
>   .../gstreamer/gstreamer1.0_git.bb                  |    2 +-
>   meta/recipes-sato/gaku/gaku_git.bb                 |    2 +-
>   .../gtk-engines/gtk-sato-engine_git.bb             |    2 +-
>   meta/recipes-sato/libowl/libowl_git.bb             |    2 +-
>   .../matchbox-config-gtk/matchbox-config-gtk_git.bb |    2 +-
>   .../matchbox-desktop-sato_git.bb                   |    2 +-
>   .../matchbox-desktop/matchbox-desktop_git.bb       |    2 +-
>   .../matchbox-keyboard/matchbox-keyboard_git.bb     |    2 +-
>   .../matchbox-panel-2/matchbox-panel-2_git.bb       |    2 +-
>   .../matchbox-terminal/matchbox-terminal_git.bb     |    2 +-
>   .../matchbox-theme-sato/matchbox-theme-sato_git.bb |    2 +-
>   .../recipes-sato/owl-video-widget/libowl-av_git.bb |    2 +-
>   .../recipes-sato/owl-video-widget/owl-video_git.bb |    2 +-
>   meta/recipes-sato/puzzles/oh-puzzles_git.bb        |    2 +-
>   meta/recipes-sato/puzzles/puzzles_r9765.bb         |    2 +-
>   .../sato-screenshot/sato-screenshot_git.bb         |    2 +-
>   .../settings-daemon/settings-daemon_git.bb         |    2 +-
>   meta/recipes-sato/web/web-webkit_git.bb            |    2 +-
>   98 files changed, 100 insertions(+), 100 deletions(-)

The world build is a strong indication that nothing was messed up.
Each change in the patch looks fine to me.

// Randy
>


-- 
# Randy MacLeod. SMTS, Linux, Wind River
Direct: 613.963.1350


  parent reply	other threads:[~2013-08-30  4:27 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-30  1:56 [PATCH 0/1] remove the unnecessary protocol parameters jackie.huang
2013-08-30  1:56 ` [PATCH 1/1] " jackie.huang
2013-08-30  4:27 ` Randy MacLeod [this message]
2013-08-30  9:22 ` [PATCH 0/1] " Martin Jansa
2013-09-02  2:01   ` jhuang0
  -- strict thread matches above, loose matches on Subject: below --
2013-09-02  2:54 jackie.huang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=52201F3D.7020607@windriver.com \
    --to=randy.macleod@windriver.com \
    --cc=jackie.huang@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.